Commit 99cd2581 authored Nov 21, 2016 by Kirtika Ruchandani Committed by Johannes Berg Dec 13, 2016

mac80211: Remove unused 'struct ieee80211_rx_status' ptr



Commit 554891e6 introduced 'struct ieee80211_rx_status' in
ieee80211_rx_h_defragment but did not use it. Compiling with W=1
gives the following warning, fix it.

net/mac80211/rx.c: In function ‘ieee80211_rx_h_defragment’:
net/mac80211/rx.c:1911:30: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]

Fixes: 554891e6 ("mac80211: move packet flags into packet")
Cc: Johannes Berg <johannes.berg@intel.com>
Cc: John W. Linville <linville@tuxdriver.com>
Signed-off-by: Kirtika Ruchandani <kirtika@google.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>

parent b528414c

net/mac80211/rx.c

+0 −4

Original line number	Diff line number	Diff line
		@@ -1908,7 +1908,6 @@ ieee80211_rx_h_defragment(struct ieee80211_rx_data *rx)
		unsigned int frag, seq;
		struct ieee80211_fragment_entry *entry;
		struct sk_buff *skb;
		struct ieee80211_rx_status *status;

		hdr = (struct ieee80211_hdr *)rx->skb->data;
		fc = hdr->frame_control;
		@@ -2034,9 +2033,6 @@ ieee80211_rx_h_defragment(struct ieee80211_rx_data *rx)
		dev_kfree_skb(skb);
		}

		/* Complete frame has been reassembled - process it now */
		status = IEEE80211_SKB_RXCB(rx->skb);

		out:
		ieee80211_led_rx(rx->local);
		out_no_led:

Admin message