Commit 8d5c6a1d authored by Martin Kaiser's avatar Martin Kaiser Committed by Greg Kroah-Hartman
Browse files

staging: r8188eu: go2asoc is not needed 



Remove the go2asoc variable in OnAuthClient and call start_clnt_assoc
directly. This makes the code a tiny bit shorter.

Signed-off-by: default avatarMartin Kaiser <martin@kaiser.cx>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Link: https://lore.kernel.org/r/20221024081417.66441-18-martin@kaiser.cx


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent d667d36d

drivers/staging/r8188eu/core/rtw_mlme_ext.c

+2 −6
Original line number Diff line number Diff line
@@ -796,7 +796,6 @@ static void OnAuthClient(struct adapter *padapter, struct recv_frame *precv_fram 
{

	unsigned int	seq, len, status, offset;

	unsigned char	*p;

	unsigned int	go2asoc = 0;

	struct mlme_ext_priv	*pmlmeext = &padapter->mlmeextpriv;

	struct mlme_ext_info	*pmlmeinfo = &pmlmeext->mlmext_info;

	u8 *pframe = precv_frame->rx_data;
@@ -843,16 +842,13 @@ static void OnAuthClient(struct adapter *padapter, struct recv_frame *precv_fram 
			return;

		} else {

			/*  open system */

			go2asoc = 1;

			start_clnt_assoc(padapter);

		}

	} else if (seq == 4) {

		if (pmlmeinfo->auth_algo == dot11AuthAlgrthm_Shared)

			go2asoc = 1;

	}



	if (go2asoc)

			start_clnt_assoc(padapter);

	}

}



static void UpdateBrateTbl(u8 *mbrate)

{