diff options
author | Julia Lawall <julia@diku.dk> | 2010-08-28 17:41:06 +0200 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2010-08-31 14:20:40 -0400 |
commit | 3653910714a4a9b19aadb202c24f7b1ae61d3556 (patch) | |
tree | 9f26e90bfa6d1d905dbdab89b80c432f4e2e6a76 /net/wireless/wext-core.c | |
parent | 9df86e2e702c6d5547aced7f241addd2d698bb11 (diff) | |
download | lwn-3653910714a4a9b19aadb202c24f7b1ae61d3556.tar.gz lwn-3653910714a4a9b19aadb202c24f7b1ae61d3556.zip |
net/wireless: Remove double test
The same expression is tested twice and the result is the same each 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: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/wireless/wext-core.c')
-rw-r--r-- | net/wireless/wext-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/wireless/wext-core.c b/net/wireless/wext-core.c index 0ef17bc42bac..40385936e286 100644 --- a/net/wireless/wext-core.c +++ b/net/wireless/wext-core.c @@ -611,7 +611,7 @@ struct iw_statistics *get_wireless_stats(struct net_device *dev) #endif #ifdef CONFIG_CFG80211_WEXT - if (dev->ieee80211_ptr && dev->ieee80211_ptr && + if (dev->ieee80211_ptr && dev->ieee80211_ptr->wiphy && dev->ieee80211_ptr->wiphy->wext && dev->ieee80211_ptr->wiphy->wext->get_wireless_stats) |