diff options
| author | Tony Battersby <tonyb@cybernetics.com> | 2007-10-11 04:09:04 (GMT) |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2007-10-11 04:09:04 (GMT) |
| commit | fa8705b00aeca19d91a1437b8a5cf865999b28f6 (patch) | |
| tree | d91cf4fe1b632836d286a1216b4f821903b8d4f5 | |
| parent | 414c66e00e9fb4572e38bd14e6cc570eae8c5a61 (diff) | |
| download | linux-fsl-qoriq-fa8705b00aeca19d91a1437b8a5cf865999b28f6.tar.xz | |
[NET]: sanitize kernel_accept() error path
If kernel_accept() returns an error, it may pass back a pointer to
freed memory (which the caller should ignore). Make it pass back NULL
instead for better safety.
Signed-off-by: Tony Battersby <tonyb@cybernetics.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/socket.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/socket.c b/net/socket.c index d233647..379b3a3 100644 --- a/net/socket.c +++ b/net/socket.c @@ -2235,6 +2235,7 @@ int kernel_accept(struct socket *sock, struct socket **newsock, int flags) err = sock->ops->accept(sock, *newsock, flags); if (err < 0) { sock_release(*newsock); + *newsock = NULL; goto done; } |