summaryrefslogtreecommitdiffstats
path: root/net/core/sock.c
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2010-03-06 01:04:45 +0000
committerDavid S. Miller <davem@davemloft.net>2010-03-07 15:25:50 -0800
commit72150e9b7fec217fbd646a29ea2f65a3d4d55ea9 (patch)
tree5c049ed5ec0642cd408286cc049917e2a0d2520d /net/core/sock.c
parentea3fb371b2a391958670f2a65e1203f7dba61671 (diff)
downloadop-kernel-dev-72150e9b7fec217fbd646a29ea2f65a3d4d55ea9.zip
op-kernel-dev-72150e9b7fec217fbd646a29ea2f65a3d4d55ea9.tar.gz
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/core/sock.c')
-rw-r--r--net/core/sock.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/net/core/sock.c b/net/core/sock.c
index 61a65a2..c5812bb 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;
OpenPOWER on IntegriCloud