diff options
author | Michal Miroslaw <mirq-linux@rere.qmqm.pl> | 2007-03-04 18:59:20 -0500 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-03-05 16:25:24 -0500 |
commit | dd16704eba171b32ef0cded3a4f562b33b911066 (patch) | |
tree | b596880077eb678110373c62b2c6c398ff2d0533 /net | |
parent | a497097d35d37b47e885cf15bcaea01f279fe5e6 (diff) |
[NETFILTER]: nfnetlink_log: fix possible NULL pointer dereference
Eliminate possible NULL pointer dereference in nfulnl_recv_config().
Signed-off-by: Michal Miroslaw <mirq-linux@rere.qmqm.pl>
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/netfilter/nfnetlink_log.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/net/netfilter/nfnetlink_log.c b/net/netfilter/nfnetlink_log.c index 1b940512d87b..b669db564695 100644 --- a/net/netfilter/nfnetlink_log.c +++ b/net/netfilter/nfnetlink_log.c | |||
@@ -858,6 +858,9 @@ nfulnl_recv_config(struct sock *ctnl, struct sk_buff *skb, | |||
858 | ret = -EINVAL; | 858 | ret = -EINVAL; |
859 | break; | 859 | break; |
860 | } | 860 | } |
861 | |||
862 | if (!inst) | ||
863 | goto out; | ||
861 | } else { | 864 | } else { |
862 | if (!inst) { | 865 | if (!inst) { |
863 | UDEBUG("no config command, and no instance for " | 866 | UDEBUG("no config command, and no instance for " |
@@ -911,6 +914,7 @@ nfulnl_recv_config(struct sock *ctnl, struct sk_buff *skb, | |||
911 | 914 | ||
912 | out_put: | 915 | out_put: |
913 | instance_put(inst); | 916 | instance_put(inst); |
917 | out: | ||
914 | return ret; | 918 | return ret; |
915 | } | 919 | } |
916 | 920 | ||