diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-03-06 04:04:45 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-03-08 02:25:50 +0300 |
| commit | 72150e9b7fec217fbd646a29ea2f65a3d4d55ea9 (patch) | |
| tree | 5c049ed5ec0642cd408286cc049917e2a0d2520d | |
| parent | ea3fb371b2a391958670f2a65e1203f7dba61671 (diff) | |
| download | linux-72150e9b7fec217fbd646a29ea2f65a3d4d55ea9.tar.xz | |
sock.c: potential null dereference
We test that "prot->rsk_prot" is non-null right before we dereference it
on this line.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/core/sock.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/core/sock.c b/net/core/sock.c index 61a65a2e0455..c5812bbc2cc9 100644 --- a/net/core/sock.c +++ b/net/core/sock.c @@ -2288,7 +2288,8 @@ out_free_request_sock_slab: prot->rsk_prot->slab = NULL; } out_free_request_sock_slab_name: - kfree(prot->rsk_prot->slab_name); + if (prot->rsk_prot) + kfree(prot->rsk_prot->slab_name); out_free_sock_slab: kmem_cache_destroy(prot->slab); prot->slab = NULL; |