Browse Source

staging: rtl8188eu: remove redundant NULL check

The check for pstat and pdvobjpriv is not required here as we have
already checked for them before.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Sudip Mukherjee 10 years ago
parent
commit
2773941984

+ 1 - 1
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c

@@ -3367,7 +3367,7 @@ static unsigned int OnAssocReq(struct adapter *padapter,
 	spin_unlock_bh(&pstapriv->asoc_list_lock);
 	spin_unlock_bh(&pstapriv->asoc_list_lock);
 
 
 	/*  now the station is qualified to join our BSS... */
 	/*  now the station is qualified to join our BSS... */
-	if (pstat && (pstat->state & WIFI_FW_ASSOC_SUCCESS) && (_STATS_SUCCESSFUL_ == status)) {
+	if ((pstat->state & WIFI_FW_ASSOC_SUCCESS) && (_STATS_SUCCESSFUL_ == status)) {
 		/* 1 bss_cap_update & sta_info_update */
 		/* 1 bss_cap_update & sta_info_update */
 		bss_cap_update_on_sta_join(padapter, pstat);
 		bss_cap_update_on_sta_join(padapter, pstat);
 		sta_info_update(padapter, pstat);
 		sta_info_update(padapter, pstat);

+ 1 - 1
drivers/staging/rtl8188eu/os_dep/usb_intf.c

@@ -123,7 +123,7 @@ static struct dvobj_priv *usb_dvobj_init(struct usb_interface *usb_intf)
 	status = _SUCCESS;
 	status = _SUCCESS;
 
 
 free_dvobj:
 free_dvobj:
-	if (status != _SUCCESS && pdvobjpriv) {
+	if (status != _SUCCESS) {
 		usb_set_intfdata(usb_intf, NULL);
 		usb_set_intfdata(usb_intf, NULL);
 		kfree(pdvobjpriv);
 		kfree(pdvobjpriv);
 		pdvobjpriv = NULL;
 		pdvobjpriv = NULL;