Commit c22afb2e authored Dec 30, 2022 by Martin Kaiser Committed by Greg Kroah-Hartman Jan 17, 2023

staging: r8188eu: return immediately if we're not meant to encrypt



In function xmitframe_swencrypt, we can return immediately if our packet
needs no encryption. This is simpler than wrapping all the code into a
large if statement.

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/20221230180646.91008-5-martin@kaiser.cx


Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 00df4073

drivers/staging/r8188eu/core/rtw_xmit.c

+16 −15

Original line number	Diff line number	Diff line
		@@ -765,7 +765,9 @@ static void xmitframe_swencrypt(struct adapter padapter, struct xmit_frame pxm
		{
		struct pkt_attrib *pattrib = &pxmitframe->attrib;

		if (pattrib->bswenc) {
		if (!pattrib->bswenc)
		return;

		switch (pattrib->encrypt) {
		case _WEP40_:
		case _WEP104_:
		@@ -781,7 +783,6 @@ static void xmitframe_swencrypt(struct adapter padapter, struct xmit_frame pxm
		break;
		}
		}
		}

		s32 rtw_make_wlanhdr(struct adapter padapter, u8 hdr, struct pkt_attrib *pattrib)
		{