staging: r8188eu: remove unnecessary else branch
authorMartin Kaiser <martin@kaiser.cx>
Mon, 24 Oct 2022 08:14:14 +0000 (10:14 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 25 Oct 2022 17:19:39 +0000 (19:19 +0200)
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>
drivers/staging/r8188eu/core/rtw_mlme_ext.c

index e2e7c35..6ac3e41 100644 (file)
@@ -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) {