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

[Alexandre Julliard]

Jason Green <jave27(a)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(a)winehq.org