summaryrefslogtreecommitdiff
path: root/net
diff options
context:
space:
mode:
authorPatrick McHardy <kaber@trash.net>2006-03-12 20:40:43 -0800
committerDavid S. Miller <davem@davemloft.net>2006-03-12 20:40:43 -0800
commit31fe4d331729e9687db84521c3ceb8e43390efcf (patch)
tree3db1ef1a3daaf10d01ab18dbcc0f107857919eac /net
parentbaa829d8926f02ab04be6ec37780810d221c5b4b (diff)
downloadlinux-3.10-31fe4d331729e9687db84521c3ceb8e43390efcf.tar.gz
linux-3.10-31fe4d331729e9687db84521c3ceb8e43390efcf.tar.bz2
linux-3.10-31fe4d331729e9687db84521c3ceb8e43390efcf.zip
[NETFILTER]: arp_tables: fix NULL pointer dereference
The check is wrong and lets NULL-ptrs slip through since !IS_ERR(NULL) is true. Coverity #190 Signed-off-by: Patrick McHardy <kaber@trash.net> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/ipv4/netfilter/arp_tables.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/netfilter/arp_tables.c b/net/ipv4/netfilter/arp_tables.c
index dd1048be8a0..7d7ab94a7a2 100644
--- a/net/ipv4/netfilter/arp_tables.c
+++ b/net/ipv4/netfilter/arp_tables.c
@@ -771,7 +771,7 @@ static int get_entries(const struct arpt_get_entries *entries,
struct arpt_table *t;
t = xt_find_table_lock(NF_ARP, entries->name);
- if (t || !IS_ERR(t)) {
+ if (t && !IS_ERR(t)) {
struct xt_table_info *private = t->private;
duprintf("t->private->number = %u\n",
private->number);