Re: [PATCH v2 1/3] wininet: Return error from HttpQueryInfo if number argument is invalid.

[Alexandre Julliard]

Daniel Lehman <dlehman25(a)gmail.com> writes:
> +        value = strtoulW( lphttpHdr->lpszValue, NULL, 10 );
> +        if (value >= UINT_MAX)
> +        {
> +            LeaveCriticalSection( &request->headers_section );
> +            return ERROR_HTTP_INVALID_HEADER;
> +        }

Is UINT_MAX really supposed to fail? Otherwise you need an errno
check. Either way, some more test cases would be a good idea.

-- 
Alexandre Julliard
julliard(a)winehq.org