diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-11-27 20:31:19 +0300 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2012-11-28 13:46:22 +0100 |
commit | 0624760c96da3ffeaec8027a016475ae752d75e1 (patch) | |
tree | ad2624cfa20c66d297f1511b96d4d192116c6f2b /net/mac80211/rx.c | |
parent | d9d8b01978f6510a262ddc4da3b3694d5b149cfe (diff) | |
download | op-kernel-dev-0624760c96da3ffeaec8027a016475ae752d75e1.zip op-kernel-dev-0624760c96da3ffeaec8027a016475ae752d75e1.tar.gz |
mac80211: fix potential NULL dereference
Smatch complains that we could dereference skb later in the function.
It's probably unlikely, but we may as well return here and avoid it.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
[change summary]
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/rx.c')
-rw-r--r-- | net/mac80211/rx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/mac80211/rx.c b/net/mac80211/rx.c index 9b13b8b..db343fa 100644 --- a/net/mac80211/rx.c +++ b/net/mac80211/rx.c @@ -49,7 +49,7 @@ static struct sk_buff *remove_monitor_info(struct ieee80211_local *local, /* driver bug */ WARN_ON(1); dev_kfree_skb(skb); - skb = NULL; + return NULL; } } |