Re: Bug in meta tag refresh code [with cause]

On 03/05/16 01:59, Jorge Arellano Cid wrote:

>   Please try to be clear & concise.

I thought I was :)

>   There's a bug, yes. You may try this:
>
> diff -r c20e74568504 src/html.cc
> --- a/src/html.cc       Sun May 01 10:49:17 2016 -0300
> +++ b/src/html.cc       Sun May 01 23:26:48 2016 -0300
> @@ -3220,6 +3220,8 @@ static void Html_tag_open_meta(DilloHtml
>              content = p + strlen("url=");
>           else if ((p = strstr(content, ";")))
>              content = p + strlen(";");
> +         else
> +            content = "";
>           /* Handle the case of a quoted URL */
>           if (*content == '"' || *content == '\'') {
>              if ((p = strchr(content + 1, *content)))
>
>
>   @Johannes, does it look OK to you?

OK, that doesn't quite work correctly - it removes the warning messages 
altogether.

Using:

content = "./";

produces the warning message and with the top level URL - but, a case 
like this:

www.example.com/refresh.html

should produce a refresh to that URL, but it drops the page and points 
to www.example.com/

Nearly there...

Nick

-- 
Gosh that takes me back... or is it forward?  That's the trouble with
time travel, you never can tell."
		-- Doctor Who "Androids of Tara"