summaryrefslogtreecommitdiffstats
path: root/lib/rhashtable.c
diff options
context:
space:
mode:
authorHerbert Xu <herbert@gondor.apana.org.au>2017-02-25 22:38:11 +0800
committerDavid S. Miller <davem@davemloft.net>2017-02-26 21:32:57 -0500
commitca435407ba6655a0d63230818ffe7006d967dfe5 (patch)
tree715d4494ea64d0e6ab26518a0c1489ed0491a134 /lib/rhashtable.c
parent37f1c63e3e29181ad412485eec2200cffc1dc70f (diff)
downloadop-kernel-dev-ca435407ba6655a0d63230818ffe7006d967dfe5.zip
op-kernel-dev-ca435407ba6655a0d63230818ffe7006d967dfe5.tar.gz
rhashtable: Fix use before NULL check in bucket_table_free
Dan Carpenter reported a use before NULL check bug in the function bucket_table_free. In fact we don't need the NULL check at all as no caller can provide a NULL argument. So this patch fixes this by simply removing it. Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'lib/rhashtable.c')
-rw-r--r--lib/rhashtable.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/lib/rhashtable.c b/lib/rhashtable.c
index 172454e..fac1a78 100644
--- a/lib/rhashtable.c
+++ b/lib/rhashtable.c
@@ -146,9 +146,7 @@ static void bucket_table_free(const struct bucket_table *tbl)
if (tbl->nest)
nested_bucket_table_free(tbl);
- if (tbl)
- kvfree(tbl->locks);
-
+ kvfree(tbl->locks);
kvfree(tbl);
}
OpenPOWER on IntegriCloud