staging: r8188eu: remove category check in OnAction_p2p
authorMartin Kaiser <martin@kaiser.cx>
Sun, 30 Oct 2022 17:33:19 +0000 (18:33 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 31 Oct 2022 08:09:03 +0000 (09:09 +0100)
The caller of OnAction_p2p has already checked the action category. We can
remove the check in OnAction_p2p.

Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20221030173326.1588647-7-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/core/rtw_mlme_ext.c

index f1054192bfb19466f43796e1c3ad11a1fce5ed3f..efcb2f3b6d3f7a494d5bfcb876c7e4042c89f9cf 100644 (file)
@@ -3835,7 +3835,7 @@ static void on_action_public(struct adapter *padapter, struct recv_frame *precv_
 static void OnAction_p2p(struct adapter *padapter, struct recv_frame *precv_frame)
 {
        u8 *frame_body;
-       u8 category, OUI_Subtype;
+       u8 OUI_Subtype;
        u8 *pframe = precv_frame->rx_data;
        uint len = precv_frame->len;
        struct  wifidirect_info *pwdinfo = &padapter->wdinfo;
@@ -3846,10 +3846,6 @@ static void OnAction_p2p(struct adapter *padapter, struct recv_frame *precv_fram
 
        frame_body = (unsigned char *)(pframe + sizeof(struct ieee80211_hdr_3addr));
 
-       category = frame_body[0];
-       if (category != RTW_WLAN_CATEGORY_P2P)
-               return;
-
        if (be32_to_cpu(*((__be32 *)(frame_body + 1))) != P2POUI)
                return;