From: Shivani Bhardwaj Date: Wed, 14 Oct 2015 10:19:06 +0000 (+0530) Subject: Staging: rtl8723au: rtw_mlme_ext: Remove unnecessary test expression X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=5aebea5590295ab35df87cc0c0a19f0d0241f012;p=openwrt%2Fstaging%2Fblogic.git Staging: rtl8723au: rtw_mlme_ext: Remove unnecessary test expression Logical and-ing of a struct and its member should be replaced by the member only as the struct has already been NULL tested before. Signed-off-by: Shivani Bhardwaj Reviewed-by: Daniel Baluta Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c index fca22ba896b3..d28f29a93810 100644 --- a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c +++ b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c @@ -1715,7 +1715,7 @@ OnAssocReq23a(struct rtw_adapter *padapter, struct recv_frame *precv_frame) spin_unlock_bh(&pstapriv->asoc_list_lock); /* now the station is qualified to join our BSS... */ - if (pstat && pstat->state & WIFI_FW_ASSOC_SUCCESS && + if (pstat->state & WIFI_FW_ASSOC_SUCCESS && status == WLAN_STATUS_SUCCESS) { /* 1 bss_cap_update & sta_info_update23a */ bss_cap_update_on_sta_join23a(padapter, pstat);