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

Alexandre Julliard julliard at winehq.org
Tue Aug 27 04:40:05 CDT 2019


Daniel Lehman <dlehman25 at gmail.com> writes:

> +        value = strtoullW( lphttpHdr->lpszValue, NULL, 10 );
> +        if (value > UINT_MAX)
> +        {
> +            LeaveCriticalSection( &request->headers_section );
> +            return ERROR_HTTP_INVALID_HEADER;
> +        }

If you are not going to return a 64-bit value, you don't need
strtoull(). strtoul() with an overflow check should be enough.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list