Commit f3a8fb07 authored by Shivani Bhardwaj's avatar Shivani Bhardwaj Committed by Greg Kroah-Hartman

Staging: rtl8192u: ieee80211_rx: Remove unnecessary code

The variable frame_authorized is declared and mentioned in the code but
is not used anywhere so, it should be removed. Also, two case statements
are merged by removing an extra break.
Semantic patch used:
@@
type T;
identifier i;
constant C;
position p != e.p;
@@

- T i@p;
  <+... when != i
- i = C;
  ...+>
Signed-off-by: default avatarShivani Bhardwaj <shivanib134@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent afd0fea4
...@@ -899,7 +899,6 @@ int ieee80211_rx(struct ieee80211_device *ieee, struct sk_buff *skb, ...@@ -899,7 +899,6 @@ int ieee80211_rx(struct ieee80211_device *ieee, struct sk_buff *skb,
struct net_device *wds = NULL; struct net_device *wds = NULL;
struct sk_buff *skb2 = NULL; struct sk_buff *skb2 = NULL;
struct net_device *wds = NULL; struct net_device *wds = NULL;
int frame_authorized = 0;
int from_assoc_ap = 0; int from_assoc_ap = 0;
void *sta = NULL; void *sta = NULL;
#endif #endif
...@@ -1106,10 +1105,7 @@ int ieee80211_rx(struct ieee80211_device *ieee, struct sk_buff *skb, ...@@ -1106,10 +1105,7 @@ int ieee80211_rx(struct ieee80211_device *ieee, struct sk_buff *skb,
switch (hostap_handle_sta_rx(ieee, dev, skb, rx_stats, switch (hostap_handle_sta_rx(ieee, dev, skb, rx_stats,
wds != NULL)) { wds != NULL)) {
case AP_RX_CONTINUE_NOT_AUTHORIZED: case AP_RX_CONTINUE_NOT_AUTHORIZED:
frame_authorized = 0;
break;
case AP_RX_CONTINUE: case AP_RX_CONTINUE:
frame_authorized = 1;
break; break;
case AP_RX_DROP: case AP_RX_DROP:
goto rx_dropped; goto rx_dropped;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment