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 | |
| parent | d9d8b01978f6510a262ddc4da3b3694d5b149cfe (diff) | |
| download | linux-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')
| -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, /* driver bug */ WARN_ON(1); dev_kfree_skb(skb); - skb = NULL; + return NULL; } } |