Re: [PATCH 3/6] ws2_32: Fix parameters checking in WSADuplicateSocket

[Alexandre Julliard]

Bruno Jesus  writes:
> - fd = get_sock_fd( s, FILE_READ_DATA, NULL );
> - if (fd == -1)
> - {
> -
>    SetLastError(WSAENOTSOCK);
> -
>    return SOCKET_ERROR;
> - }
> - release_sock_fd( s, fd );

You should avoid retrieving the fd if you are not using it. The handle
should be checked at the point where you are doing something with it.