On Thu, 4 Oct 2007, Tony Battersby wrote:

> If accept() returns an error, kernel_accept() releases the new socket
> but passes a pointer to the released socket back to the caller.  Make it
> pass back NULL instead.
> 
> Signed-off-by: Tony Battersby <[EMAIL PROTECTED]>
> ---
> --- linux-2.6.23-rc9/net/socket.c.bak 2007-10-04 15:21:17.000000000 -0400
> +++ linux-2.6.23-rc9/net/socket.c     2007-10-04 15:21:22.000000000 -0400
> @@ -2230,6 +2230,7 @@ int kernel_accept(struct socket *sock, s
>       err = sock->ops->accept(sock, *newsock, flags);
>       if (err < 0) {
>               sock_release(*newsock);
> +             *newsock = NULL;
>               goto done;
>       }
>  

If you get an error back from kernel_accept, you should not be trying to 
use newsock.

-- 
James Morris
<[EMAIL PROTECTED]>
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to