diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-11-27 12:31:19 -0500 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2012-11-28 07:46:22 -0500 |
commit | 0624760c96da3ffeaec8027a016475ae752d75e1 (patch) | |
tree | ad2624cfa20c66d297f1511b96d4d192116c6f2b /net/mac80211/rx.c | |
parent | d9d8b01978f6510a262ddc4da3b3694d5b149cfe (diff) |
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 9b13b8b24245..db343fa8033c 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, | |||
49 | /* driver bug */ | 49 | /* driver bug */ |
50 | WARN_ON(1); | 50 | WARN_ON(1); |
51 | dev_kfree_skb(skb); | 51 | dev_kfree_skb(skb); |
52 | skb = NULL; | 52 | return NULL; |
53 | } | 53 | } |
54 | } | 54 | } |
55 | 55 | ||