[3/3] wininet: Add missing empty line to end of headers list

Alexandre Julliard julliard at winehq.org
Fri Jun 5 05:21:25 CDT 2009


Jason Green <jave27 at gmail.com> writes:

> @@ -4265,6 +4265,24 @@ static INT HTTP_GetResponseHeaders(LPWININETHTTPREQW lpwhr, BOOL clear)
>  	}
>      }while(1);
>  
> +    /* make sure the response header is terminated with an empty line.  Some apps really
> +       truly care about that empty line being there for some reason.  If it's not present,
> +       just add it to the header. */
> +    if (cchRawHeaders >= 4 && 
> +        (strcmpW(&lpszRawHeaders[cchRawHeaders - 4], szCrLf) || 
> +         strcmpW(&lpszRawHeaders[cchRawHeaders - 2], szCrLf)))
> +    {

This check doesn't seem necessary, I don't see how you could already
have an empty line in the buffer since we only store valid headers.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list