mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-02 16:55:16 -04:00
staging: r8188eu: remove duplicate category check
The caller of on_action_public has already checked the action category. We can remove the check in on_action_public. 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-3-martin@kaiser.cx Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
5c4fb46e91
commit
5998e31922
@@ -3819,16 +3819,12 @@ unsigned int on_action_public(struct adapter *padapter, struct recv_frame *precv
|
||||
unsigned int ret = _FAIL;
|
||||
u8 *pframe = precv_frame->rx_data;
|
||||
u8 *frame_body = pframe + sizeof(struct ieee80211_hdr_3addr);
|
||||
u8 category, action;
|
||||
u8 action;
|
||||
|
||||
/* check RA matches or not */
|
||||
if (memcmp(myid(&padapter->eeprompriv), mgmt->da, ETH_ALEN))
|
||||
goto exit;
|
||||
|
||||
category = frame_body[0];
|
||||
if (category != WLAN_CATEGORY_PUBLIC)
|
||||
goto exit;
|
||||
|
||||
action = frame_body[1];
|
||||
switch (action) {
|
||||
case ACT_PUBLIC_VENDOR:
|
||||
|
||||
Reference in New Issue
Block a user