Commit 8f5193f8 authored by Shreyansh Chouhan's avatar Shreyansh Chouhan Committed by Greg Kroah-Hartman
Browse files

staging: rtl8723bs: remove unnecessary braces from conditionals



Removed the braces from if else statements in core/rtw_wlan_util.c since
the previous commit (6a257dd6de516573) caused all conditional blocks to
have a single statement in the function check_assoc_AP.

Signed-off-by: default avatarShreyansh Chouhan <chouhan.shreyansh630@gmail.com>
Link: https://lore.kernel.org/r/20210524135105.5550-3-chouhan.shreyansh630@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 3c8c577f
Loading
Loading
Loading
Loading
+17 −18
Original line number Diff line number Diff line
@@ -1532,25 +1532,24 @@ unsigned char check_assoc_AP(u8 *pframe, uint len)

		switch (pIE->ElementID) {
		case WLAN_EID_VENDOR_SPECIFIC:
			if ((!memcmp(pIE->data, ARTHEROS_OUI1, 3)) || (!memcmp(pIE->data, ARTHEROS_OUI2, 3))) {
			if ((!memcmp(pIE->data, ARTHEROS_OUI1, 3)) || (!memcmp(pIE->data, ARTHEROS_OUI2, 3)))
				return HT_IOT_PEER_ATHEROS;
			} else if ((!memcmp(pIE->data, BROADCOM_OUI1, 3)) ||
			else if ((!memcmp(pIE->data, BROADCOM_OUI1, 3)) ||
			         (!memcmp(pIE->data, BROADCOM_OUI2, 3)) ||
				   (!memcmp(pIE->data, BROADCOM_OUI3, 3))) {
			         (!memcmp(pIE->data, BROADCOM_OUI3, 3)))
			      return HT_IOT_PEER_BROADCOM;
			} else if (!memcmp(pIE->data, MARVELL_OUI, 3)) {
			else if (!memcmp(pIE->data, MARVELL_OUI, 3))
			      return HT_IOT_PEER_MARVELL;
			} else if (!memcmp(pIE->data, RALINK_OUI, 3)) {
			else if (!memcmp(pIE->data, RALINK_OUI, 3))
			      return HT_IOT_PEER_RALINK;
			} else if (!memcmp(pIE->data, CISCO_OUI, 3)) {
			else if (!memcmp(pIE->data, CISCO_OUI, 3))
			      return HT_IOT_PEER_CISCO;
			} else if (!memcmp(pIE->data, REALTEK_OUI, 3)) {
			else if (!memcmp(pIE->data, REALTEK_OUI, 3))
			      return get_realtek_assoc_AP_vender(pIE);
			} else if (!memcmp(pIE->data, AIRGOCAP_OUI, 3)) {
			else if (!memcmp(pIE->data, AIRGOCAP_OUI, 3))
			      return HT_IOT_PEER_AIRGO;
			} else {
			else
			      break;
			}

		default:
			break;