On Sun, Sep 12, 2010 at 5:23 PM, Vincent Povirk <vinc...@codeweavers.com> wrote:
>
>
>
>
>
>

> @@ -3193,6 +3193,10 @@ int WINAPI WS_select(int nfds, WS_fd_set *ws_readfds,
>          return SOCKET_ERROR;
>      }
>
> +    for (i=0; i<count; i++)
> +        if (pollfds[i].fd == -1)
> +            return SOCKET_ERROR;
You're leaking pollfds here. (As a nitpick - braces for for's that
contain more than one line are nice)

Mike.


Reply via email to