diff options
author | Julia Lawall <julia@diku.dk> | 2010-08-28 11:41:00 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-08-31 18:22:37 -0400 |
commit | 4498dbcd2d85b67e9f46790bcfee5860d2dd1762 (patch) | |
tree | 3007d7515ef931678114abe946f6fee568a9acf3 /drivers/staging/rtl8192e/ieee80211 | |
parent | 15300c1df7e8c85cfbd5155a59cd6f767af70daa (diff) |
staging: rtl8193*: Remove double test
The 1 element of the array is tested twice. Change the code so that the
remaining 3 element of the array is tested instead of testing the 1 element
a second time.
The sematic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@expression@
expression E;
@@
(
* E
|| ... || E
|
* E
&& ... && E
)
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/rtl8192e/ieee80211')
-rw-r--r-- | drivers/staging/rtl8192e/ieee80211/ieee80211_rx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/rtl8192e/ieee80211/ieee80211_rx.c b/drivers/staging/rtl8192e/ieee80211/ieee80211_rx.c index aaf9b9dc45e..9318695042f 100644 --- a/drivers/staging/rtl8192e/ieee80211/ieee80211_rx.c +++ b/drivers/staging/rtl8192e/ieee80211/ieee80211_rx.c | |||
@@ -2579,7 +2579,7 @@ static inline void update_network(struct ieee80211_network *dst, | |||
2579 | if(src->wmm_param[0].ac_aci_acm_aifsn|| \ | 2579 | if(src->wmm_param[0].ac_aci_acm_aifsn|| \ |
2580 | src->wmm_param[1].ac_aci_acm_aifsn|| \ | 2580 | src->wmm_param[1].ac_aci_acm_aifsn|| \ |
2581 | src->wmm_param[2].ac_aci_acm_aifsn|| \ | 2581 | src->wmm_param[2].ac_aci_acm_aifsn|| \ |
2582 | src->wmm_param[1].ac_aci_acm_aifsn) { | 2582 | src->wmm_param[3].ac_aci_acm_aifsn) { |
2583 | memcpy(dst->wmm_param, src->wmm_param, WME_AC_PRAM_LEN); | 2583 | memcpy(dst->wmm_param, src->wmm_param, WME_AC_PRAM_LEN); |
2584 | } | 2584 | } |
2585 | //dst->QoS_Enable = src->QoS_Enable; | 2585 | //dst->QoS_Enable = src->QoS_Enable; |