From: Martin Kaiser <martin@kaiser.cx>
Date: Mon, 24 Oct 2022 08:14:14 +0000 (+0200)
Subject: staging: r8188eu: remove unnecessary else branch
X-Git-Tag: v6.2-rc1~69^2~159
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=3cf90ead512362a1b2b4b9a0d48ec47ca26744c2;p=thirdparty%2Fkernel%2Flinux.git

staging: r8188eu: remove unnecessary else branch

Remove an else branch in OnAuthClient that is not needed.

If we go into the else branch, go2asoc is 0. We can simply continue and
the last if condition will be false.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Link: https://lore.kernel.org/r/20221024081417.66441-15-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
index e2e7c35019f2e..6ac3e41af992f 100644
--- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c
+++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
@@ -850,9 +850,6 @@ static void OnAuthClient(struct adapter *padapter, struct recv_frame *precv_fram
 			go2asoc = 1;
 		else
 			return;
-	} else {
-		/*  this is also illegal */
-		return;
 	}
 
 	if (go2asoc) {