diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-03-06 02:04:45 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-03-08 00:25:50 +0100 |
| commit | 72150e9b7fec217fbd646a29ea2f65a3d4d55ea9 (patch) | |
| tree | 5c049ed5ec0642cd408286cc049917e2a0d2520d /net | |
| parent | ems_usb: cleanup: remove uneeded check (diff) | |
| download | linux-72150e9b7fec217fbd646a29ea2f65a3d4d55ea9.tar.xz linux-72150e9b7fec217fbd646a29ea2f65a3d4d55ea9.zip | |
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>
Diffstat (limited to '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; |