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.

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

[Bruno Jesus]

On Mon, Sep 9, 2013 at 6:39 AM, Alexandre Julliard  wrote:
> 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.

I don't need the the handle, I just need to be sure that is a valid
socket. The first parameters checked is the socket. I need to find a
way to do that and keep the tests working. What if I use a
getsockopt(SO_TYPE) ?
> --
> Alexandre Julliard
> [e-mail]

Regards,
Bruno

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

[Alexandre Julliard]

Bruno Jesus  writes:
> On Mon, Sep 9, 2013 at 6:39 AM, Alexandre Julliard  wrote:
> > 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.
>
> I don't need the the handle, I just need to be sure that is a valid
> socket. The first parameters checked is the socket. I need to find a
> way to do that and keep the tests working. What if I use a
> getsockopt(SO_TYPE) ?

You'll have to do something with the socket (get the protocol), so the
check should happen at that point.