Re: [6/6] wininet: Handle INTERNET_INVALID_PORT_NUMBER in HttpOpenRequest.

[Jacek Caban]

Hi Sebastian,

On 04/25/16 21:46, Sebastian Lackner wrote:
> - request->server = get_server(session->hostName, session->hostPort, (dwFlags & INTERNET_FLAG_SECURE) != 0, TRUE);
> - port = session->hostPort;
> - if (port == INTERNET_INVALID_PORT_NUMBER)
> - {
> -
>    port = (session->hdr.dwFlags & INTERNET_FLAG_SECURE) ?
> -
>            INTERNET_DEFAULT_HTTPS_PORT : INTERNET_DEFAULT_HTTP_PORT;
> - }
> -
> - request->server = get_server(session->hostName, port, (dwFlags & INTERNET_FLAG_SECURE) != 0, TRUE);

Note that get_server already has code to deal with
INTERNET_INVALID_PORT_NUMBER, so it would be enough to handle https case
there. However, given that all other callers have valid port numbers, I
agree that it would be cleaner to do that in HTTP_HttpOpenRequestW
instead. Given that your moving the logic around, please remove no
longer needed code there.

Thanks,
Jacek