skyfall/rtl8188eu
1
0
Fork 0
mirror of https://github.com/lwfinger/rtl8188eu.git synced 2024-11-25 22:13:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

rtl8188eu: Fix Sparse problems in core/rtw_mlme_ext.c

Browse source 
Sparse reports the following:

  CHECK   /home/finger/rtl8188eu/core/rtw_mlme_ext.c
/home/finger/rtl8188eu/core/rtw_mlme_ext.c:9446:6: warning: symbol '_linked_rx_signal_strehgth_display' was not declared. Should it be static?
/home/finger/rtl8188eu/core/rtw_mlme_ext.c:8520:6: warning: context imbalance in 'start_clnt_join' - different lock contexts for basic block

The second warning was due to the failure to unlock a spin lock in a error
path.

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
This commit is contained in:
Larry Finger 2015-08-14 22:56:49 -05:00
parent efddeb40e2
commit cc88df7da3
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
core/rtw_mlme_ext.c
View file

@ -8562,6 +8562,7 @@ void start_clnt_join(struct adapter* padapter)
scanned = LIST_CONTAINOR(pos, struct wlan_network, list); scanned = LIST_CONTAINOR(pos, struct wlan_network, list);
if(scanned==NULL) { if(scanned==NULL) {
spin_unlock_bh(&(padapter->mlmepriv.scanned_queue.lock));
rtw_warn_on(1); rtw_warn_on(1);
return; return;
} }
@ -9443,7 +9444,7 @@ void mlmeext_sta_del_event_callback(struct adapter *padapter)
Following are the functions for the timer handlers Following are the functions for the timer handlers
*****************************************************************************/ *****************************************************************************/
void _linked_rx_signal_strehgth_display(struct adapter *padapter) static void _linked_rx_signal_strehgth_display(struct adapter *padapter)
{ {
struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv; struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info); struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);