From nobody Fri Sep 25 09:20:34 2026 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 46FE647A894 for ; Mon, 14 Sep 2026 14:53:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789397601; cv=none; b=m6NvhymwcB0czY3F5fg7TooJ/cgad8KF4dLgavdWRWC6aIddv+QeKSzr/LLIswZ+47bQAzGf1O+CfSQw8fkQu1/BtwJevlTtMAe8NuNdpFzGM34r6obE6svJssXC55J/Pa7WQS0UVuPK1VRUWfRAFF1Vm0ASNb8l38S2fukiZow= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789397601; c=relaxed/simple; bh=N5IqWztkSD6d6jT8YO/ZSljhY36WERGkX/mWHbGSs8o=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version:Content-Type; b=sT5uM2oPXwXr8OtCOY4/g8sLhXwa5v+c5LT64wXEokGLMDR0e07EgbKsn66vl/EXaEA6TFzQ3KPQrBJC/7zx0I8BMoJfevgj19B0u6t/8vFd0RJHl+IPLfPPWoShxvfrPPVwyFE84HjdHsvKgyMffFI8fUtyEg2cCg3+79sJAQg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=R8cnCTZU; arc=none smtp.client-ip=209.85.128.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="R8cnCTZU" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-49e73e541d7so2458655e9.0 for ; Mon, 14 Sep 2026 07:53:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789397596; x=1790002396; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:message-id:date :subject:cc:to:from:from:to:cc:subject:date:message-id:reply-to :content-type; bh=Vi+kgOEwQ6B298JA+n1SDDNCDc6Yrj8qSoia7I1Zars=; b=R8cnCTZUIC5fTZ4g2OWHqcn6whywTygRVbKqIZCpc85wtplUOr0s8yEY/86wn55Z8p jp7W9C64wZvHnQ7EUxhNidnj7/dIhMpmIzWDdb3Ncp32Xf/1SZrfS2VoHTaDq9TVZG9w t49C0NqQbWV5PWlnUGql6mNiCbo00vbOgKMRs7QY1+0hcBwhSdK2k9vxwXA0gY6TqnXK +tNfwulZLCTDnQOpcL107kmqezlBcNAygo2O+7g+e1iUA6BGxzJ50+/wEG4pZSinwIHO vFXvtZMGYqWsgHWE9eqfiyX7YEx7LkDF7iGyzTtKtsRBQeFFhV6hbGWQX0KanuHCM3Wq EyGA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1789397596; x=1790002396; h=content-transfer-encoding:content-type:mime-version:message-id:date :subject:cc:to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to:content-type; bh=Vi+kgOEwQ6B298JA+n1SDDNCDc6Yrj8qSoia7I1Zars=; b=CNfgP3jYp1i0uJh8idKh3pofDGLvYhgHkNMKw2xsgOD60SgVXg9yYtvM8gkIXJ87Of mj77DEoYZK0pOmpr/AvUHZlgA8NM9hptCHX8/sil9zfRGx8p/A/UpWGNNbAHRkJ/UBF5 3vSHO5ZnhZKMaezVp2e8GP/sU4EX8Gj9IlheO5r7qT+a2jW/0s5tjcF4EfBz0x269bFR cfYLezKF3kn+6B+BMa1WAzZEixowoQRgFCfeUOJWN9dS37cIzQCjTvw2ETYiEjotdtxT MC/E3RUML410CeYSjDbT1wDChiYUyGwqv0yiFZB8am4EVBdYVj8PFnhuIPAPAAedm8c9 IYJQ== X-Forwarded-Encrypted: i=1; AKwUvByLMBjZgBCT1bqBT5QMmM5QfPJDbv9UxodOB1pK2BYF6HFD3W4aY+J0OdWj53YSK2GRRg7GpnxVy34Fuxk=@vger.kernel.org X-Gm-Message-State: AFuF++k7JA1mFv3317EpIWY/x01YxUHa4K28RpuNNWAF6nyac268MvuZ 9W4eXuarTz2AXhaXA+GNPa+82sgmB8UhydswbcTgFve1Ddfga/5595NP X-Gm-Gg: AYBFou0OrR+m9Xi5eoe8efgrNEJC8hzI3kpR/WfBd9e62ubm3+gzfnrnLLDXeChCEtJ 1lv5tNK1xRNqHgztYKSbcB/LaDvVp+E/rxoeijF70QrM4ViccTF9w9Xj/hDtGRqxeTNkDuJKW9X ia2yOZw6I6MNMzbPC7Ap07Em9yJ9H7+7n2B7+hLpAsXFr7cbyZmMw0B83KyCqgBGS20FDUyaEA0 /6k6SPz478Dl0gilJgpu933SLgyzUhMTJUPZsocgl0xiqDlP+9gqNCRBUAagrNl+tw8rXETDv3x PhdkI1BO8+j37peXNXzJxq8XbasMdgmh6r/vKU1rxkEUXtARM9Hvc8/4coxr1TWhocEWuJBSFro eFrQ0Woige+B2Jc+4Ny0t2oTHYtPVntiU3jMiP8vmlrJEDCwHXeV8eIQC6fmdM7BbrZXlN6F9rC jTDzoH3F008ZLCy5qfkiMQXI63d/Hw0XC2J+6LcaRzYhS3GRLWVigiasMudEuaCJxjfsKEDs9zi kdDTU/On8FOeStU41J4ag== X-Received: by 2002:a7b:cb87:0:b0:49e:718f:4f2e with SMTP id 5b1f17b1804b1-49e76021236mr55872585e9.3.1789397596079; Mon, 14 Sep 2026 07:53:16 -0700 (PDT) Received: from ubuntu-4gb-nbg1-1.. ([2a01:4f8:1c19:b0ad::1]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-486eb32e6f1sm26812525f8f.6.2026.09.14.07.53.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 14 Sep 2026 07:53:15 -0700 (PDT) From: =?UTF-8?q?Rare=C8=99-Mihai=20Vladu?= To: gregkh@linuxfoundation.org Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, =?UTF-8?q?Rare=C8=99-Mihai=20Vladu?= Subject: [PATCH] staging: rtl8723bs: fix lines exceeding 100 columns in rtw_security.c Date: Mon, 14 Sep 2026 14:52:56 +0000 Message-ID: <20260914145256.996968-1-vladurares@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Reformat long lines in rtw_security.c to fit within the 100-column limit by splitting function arguments, expressions, and moving Signed-off-by: Rare=C8=99-Mihai Vladu --- drivers/staging/rtl8723bs/core/rtw_security.c | 94 +++++++++++++------ 1 file changed, 66 insertions(+), 28 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_security.c b/drivers/stagin= g/rtl8723bs/core/rtw_security.c index 2217cffac..208aa0f31 100644 --- a/drivers/staging/rtl8723bs/core/rtw_security.c +++ b/drivers/staging/rtl8723bs/core/rtw_security.c @@ -34,7 +34,8 @@ const char *security_type_str(u8 value) =20 /* Need to consider the fragment situation */ void rtw_wep_encrypt(struct adapter *padapter, u8 *pxmitframe) -{ /* exclude ICV */ +{ + /* exclude ICV */ union { __le32 f0; unsigned char f1[4]; @@ -64,12 +65,17 @@ void rtw_wep_encrypt(struct adapter *padapter, u8 *pxmi= tframe) for (curfragnum =3D 0; curfragnum < pattrib->nr_frags; curfragnum++) { iv =3D pframe + pattrib->hdrlen; memcpy(&wepkey[0], iv, 3); - memcpy(&wepkey[3], &psecuritypriv->dot11DefKey[psecuritypriv->dot11Priv= acyKeyIndex].skey[0], keylength); + int idx =3D psecuritypriv->dot11PrivacyKeyIndex; + + memcpy(&wepkey[3], + &psecuritypriv->dot11DefKey[idx].skey[0], + keylength); payload =3D pframe + pattrib->iv_len + pattrib->hdrlen; =20 if ((curfragnum + 1) =3D=3D pattrib->nr_frags) { /* the last fragment */ =20 - length =3D pattrib->last_txcmdsz - pattrib->hdrlen - pattrib->iv_len -= pattrib->icv_len; + length =3D pattrib->last_txcmdsz - pattrib->hdrlen - + pattrib->iv_len - pattrib->icv_len; =20 crc.f0 =3D cpu_to_le32(~crc32_le(~0, payload, length)); =20 @@ -78,7 +84,8 @@ void rtw_wep_encrypt(struct adapter *padapter, u8 *pxmitf= rame) arc4_crypt(ctx, payload + length, crc.f1, 4); =20 } else { - length =3D pxmitpriv->frag_len - pattrib->hdrlen - pattrib->iv_len - p= attrib->icv_len; + length =3D pxmitpriv->frag_len - pattrib->hdrlen - + pattrib->iv_len - pattrib->icv_len; crc.f0 =3D cpu_to_le32(~crc32_le(~0, payload, length)); arc4_setkey(ctx, wepkey, 3 + keylength); arc4_crypt(ctx, payload, payload, length); @@ -112,9 +119,16 @@ void rtw_wep_decrypt(struct adapter *padapter, u8 *pr= ecvframe) keyindex =3D prxattrib->key_index; keylength =3D psecuritypriv->dot11DefKeylen[keyindex]; memcpy(&wepkey[0], iv, 3); - /* memcpy(&wepkey[3], &psecuritypriv->dot11DefKey[psecuritypriv->dot11Pr= ivacyKeyIndex].skey[0], keylength); */ - memcpy(&wepkey[3], &psecuritypriv->dot11DefKey[keyindex].skey[0], keylen= gth); - length =3D ((union recv_frame *)precvframe)->u.hdr.len - prxattrib->hdrl= en - prxattrib->iv_len; + /* + * memcpy(&wepkey[3], + * &psecuritypriv->dot11DefKey[psecuritypriv->dot11PrivacyKeyInd= ex].skey[0], + * keylength); + */ + memcpy(&wepkey[3], + &psecuritypriv->dot11DefKey[keyindex].skey[0], + keylength); + length =3D ((union recv_frame *)precvframe)->u.hdr.len - + prxattrib->hdrlen - prxattrib->iv_len; =20 payload =3D pframe + prxattrib->iv_len + prxattrib->hdrlen; =20 @@ -157,7 +171,8 @@ void rtw_secmicappendbyte(struct mic_data *pmicdata, u8= b) pmicdata->L ^=3D pmicdata->M; pmicdata->R ^=3D ROL32(pmicdata->L, 17); pmicdata->L +=3D pmicdata->R; - pmicdata->R ^=3D ((pmicdata->L & 0xff00ff00) >> 8) | ((pmicdata->L & 0x0= 0ff00ff) << 8); + pmicdata->R ^=3D ((pmicdata->L & 0xff00ff00) >> 8) | + ((pmicdata->L & 0x00ff00ff) << 8); pmicdata->L +=3D pmicdata->R; pmicdata->R ^=3D ROL32(pmicdata->L, 3); pmicdata->L +=3D pmicdata->R; @@ -418,7 +433,8 @@ static void phase2(u8 *rc4key, const u8 *tk, const u16 = *p1k, u16 iv16) =20 /* The hlen isn't include the IV */ u32 rtw_tkip_encrypt(struct adapter *padapter, u8 *pxmitframe) -{ /* exclude ICV */ +{ + /* exclude ICV */ u16 pnl; u32 pnh; u8 rc4key[16]; @@ -467,7 +483,8 @@ u32 rtw_tkip_encrypt(struct adapter *padapter, u8 *pxmi= tframe) phase2(&rc4key[0], prwskey, (u16 *)&ttkey[0], pnl); =20 if ((curfragnum + 1) =3D=3D pattrib->nr_frags) { /* 4 the last fragment = */ - length =3D pattrib->last_txcmdsz - pattrib->hdrlen - pattrib->iv_len - = pattrib->icv_len; + length =3D pattrib->last_txcmdsz - pattrib->hdrlen - + pattrib->iv_len - pattrib->icv_len; crc.f0 =3D cpu_to_le32(~crc32_le(~0, payload, length)); =20 arc4_setkey(ctx, rc4key, 16); @@ -475,7 +492,8 @@ u32 rtw_tkip_encrypt(struct adapter *padapter, u8 *pxmi= tframe) arc4_crypt(ctx, payload + length, crc.f1, 4); =20 } else { - length =3D pxmitpriv->frag_len - pattrib->hdrlen - pattrib->iv_len - pa= ttrib->icv_len; + length =3D pxmitpriv->frag_len - pattrib->hdrlen - + pattrib->iv_len - pattrib->icv_len; crc.f0 =3D cpu_to_le32(~crc32_le(~0, payload, length)); =20 arc4_setkey(ctx, rc4key, 16); @@ -491,7 +509,8 @@ u32 rtw_tkip_encrypt(struct adapter *padapter, u8 *pxmi= tframe) =20 /* The hlen isn't include the IV */ u32 rtw_tkip_decrypt(struct adapter *padapter, u8 *precvframe) -{ /* exclude ICV */ +{ + /* exclude ICV */ u16 pnl; u32 pnh; u8 rc4key[16]; @@ -536,7 +555,8 @@ u32 rtw_tkip_decrypt(struct adapter *padapter, u8 *prec= vframe) if (jiffies_to_msecs(jiffies - start) > 1000) { if (no_gkey_bc_cnt || no_gkey_mc_cnt) { netdev_dbg(padapter->pnetdev, - FUNC_ADPT_FMT " no_gkey_bc_cnt:%u, no_gkey_mc_cnt:%u\n", + FUNC_ADPT_FMT + " no_gkey_bc_cnt:%u, no_gkey_mc_cnt:%u\n", FUNC_ADPT_ARG(padapter), no_gkey_bc_cnt, no_gkey_mc_cnt); @@ -550,7 +570,8 @@ u32 rtw_tkip_decrypt(struct adapter *padapter, u8 *prec= vframe) =20 if (no_gkey_bc_cnt || no_gkey_mc_cnt) { netdev_dbg(padapter->pnetdev, - FUNC_ADPT_FMT " gkey installed. no_gkey_bc_cnt:%u, no_gkey_mc_cnt:%= u\n", + FUNC_ADPT_FMT + " gkey installed. no_gkey_bc_cnt:%u, no_gkey_mc_cnt:%u\n", FUNC_ADPT_ARG(padapter), no_gkey_bc_cnt, no_gkey_mc_cnt); @@ -566,7 +587,8 @@ u32 rtw_tkip_decrypt(struct adapter *padapter, u8 *prec= vframe) =20 iv =3D pframe + prxattrib->hdrlen; payload =3D pframe + prxattrib->iv_len + prxattrib->hdrlen; - length =3D ((union recv_frame *)precvframe)->u.hdr.len - prxattrib->hdrle= n - prxattrib->iv_len; + length =3D ((union recv_frame *)precvframe)->u.hdr.len - + prxattrib->hdrlen - prxattrib->iv_len; =20 GET_TKIP_PN(iv, dot11txpn); =20 @@ -757,10 +779,12 @@ static void construct_ctr_preload(u8 *ctr_preload, if (frtype =3D=3D WIFI_MGT_TYPE) ctr_preload[1] |=3D BIT(4); =20 + /* ctr_preload[2:7] =3D A2[0:5] =3D mpdu[10:15] */ for (i =3D 2; i < 8; i++) - ctr_preload[i] =3D mpdu[i + 8]; /* ctr_preload[2:7= ] =3D A2[0:5] =3D mpdu[10:15] */ + ctr_preload[i] =3D mpdu[i + 8]; + /* ctr_preload[8:13] =3D PN[5:0] */ for (i =3D 8; i < 14; i++) - ctr_preload[i] =3D pn_vector[13 - i]; /* ctr_preload[8:13] = =3D PN[5:0] */ + ctr_preload[i] =3D pn_vector[13 - i]; ctr_preload[14] =3D (unsigned char)(c / 256); /* Ctr */ ctr_preload[15] =3D (unsigned char)(c % 256); } @@ -952,11 +976,13 @@ u32 rtw_aes_encrypt(struct adapter *padapter, u8 *pxm= itframe) =20 for (curfragnum =3D 0; curfragnum < pattrib->nr_frags; curfragnum++) { if ((curfragnum + 1) =3D=3D pattrib->nr_frags) { /* 4 the last fragment = */ - length =3D pattrib->last_txcmdsz - pattrib->hdrlen - pattrib->iv_len - = pattrib->icv_len; + length =3D pattrib->last_txcmdsz - pattrib->hdrlen - + pattrib->iv_len - pattrib->icv_len; =20 aes_cipher(prwskey, pattrib->hdrlen, pframe, length); } else { - length =3D pxmitpriv->frag_len - pattrib->hdrlen - pattrib->iv_len - pa= ttrib->icv_len; + length =3D pxmitpriv->frag_len - pattrib->hdrlen - + pattrib->iv_len - pattrib->icv_len; =20 aes_cipher(prwskey, pattrib->hdrlen, pframe, length); pframe +=3D pxmitpriv->frag_len; @@ -1205,7 +1231,8 @@ u32 rtw_aes_decrypt(struct adapter *padapter, u8 *pre= cvframe) if (jiffies_to_msecs(jiffies - start) > 1000) { if (no_gkey_bc_cnt || no_gkey_mc_cnt) { netdev_dbg(padapter->pnetdev, - FUNC_ADPT_FMT " no_gkey_bc_cnt:%u, no_gkey_mc_cnt:%u\n", + FUNC_ADPT_FMT + " no_gkey_bc_cnt:%u, no_gkey_mc_cnt:%u\n", FUNC_ADPT_ARG(padapter), no_gkey_bc_cnt, no_gkey_mc_cnt); @@ -1220,7 +1247,8 @@ u32 rtw_aes_decrypt(struct adapter *padapter, u8 *pre= cvframe) =20 if (no_gkey_bc_cnt || no_gkey_mc_cnt) { netdev_dbg(padapter->pnetdev, - FUNC_ADPT_FMT " gkey installed. no_gkey_bc_cnt:%u, no_gkey_mc_cnt:%= u\n", + FUNC_ADPT_FMT + " gkey installed. no_gkey_bc_cnt:%u, no_gkey_mc_cnt:%u\n", FUNC_ADPT_ARG(padapter), no_gkey_bc_cnt, no_gkey_mc_cnt); @@ -1238,7 +1266,8 @@ u32 rtw_aes_decrypt(struct adapter *padapter, u8 *pre= cvframe) prwskey =3D &stainfo->dot118021x_UncstKey.skey[0]; } =20 - length =3D ((union recv_frame *)precvframe)->u.hdr.len - prxattrib->hdrle= n - prxattrib->iv_len; + length =3D ((union recv_frame *)precvframe)->u.hdr.len - + prxattrib->hdrlen - prxattrib->iv_len; =20 res =3D aes_decipher(prwskey, prxattrib->hdrlen, pframe, length); =20 @@ -1269,9 +1298,14 @@ u32 rtw_BIP_verify(struct adapter *padapter, u8 *pre= cvframe) /* mapping to wlan header */ pwlanhdr =3D (struct ieee80211_hdr *)pframe; /* save the frame body + MME */ - memcpy(BIP_AAD + BIP_AAD_SIZE, pframe + WLAN_HDR_A3_LEN, pattrib->pkt_len= - WLAN_HDR_A3_LEN); + memcpy(BIP_AAD + BIP_AAD_SIZE, + pframe + WLAN_HDR_A3_LEN, + pattrib->pkt_len - WLAN_HDR_A3_LEN); /* find MME IE pointer */ - p =3D rtw_get_ie(BIP_AAD + BIP_AAD_SIZE, WLAN_EID_MMIE, &len, pattrib->pk= t_len - WLAN_HDR_A3_LEN); + p =3D rtw_get_ie(BIP_AAD + BIP_AAD_SIZE, + WLAN_EID_MMIE, + &len, + pattrib->pkt_len - WLAN_HDR_A3_LEN); /* Baron */ if (p) { u16 keyid =3D 0; @@ -1300,8 +1334,10 @@ u32 rtw_BIP_verify(struct adapter *padapter, u8 *pre= cvframe) /* conscruct AAD, copy address 1 to address 3 */ memcpy(BIP_AAD + 2, &pwlanhdr->addrs, sizeof(pwlanhdr->addrs)); =20 - if (omac1_aes_128(padapter->securitypriv.dot11wBIPKey[padapter->security= priv.dot11wBIPKeyid].skey - , BIP_AAD, ori_len, mic)) + int keyid =3D padapter->securitypriv.dot11wBIPKeyid; + u8 *bip_key =3D padapter->securitypriv.dot11wBIPKey[keyid].skey; + + if (omac1_aes_128(bip_key, BIP_AAD, ori_len, mic)) goto BIP_exit; =20 /* MIC field should be last 8 bytes of packet (packet without FCS) */ @@ -1430,7 +1466,8 @@ void rtw_sec_restore_wep_key(struct adapter *adapter) struct security_priv *securitypriv =3D &adapter->securitypriv; signed int keyid; =20 - if ((securitypriv->dot11_privacy_algrthm =3D=3D _WEP40_) || (securitypriv= ->dot11_privacy_algrthm =3D=3D _WEP104_)) { + if ((securitypriv->dot11_privacy_algrthm =3D=3D _WEP40_) || + (securitypriv->dot11_privacy_algrthm =3D=3D _WEP104_)) { for (keyid =3D 0; keyid < 4; keyid++) { if (securitypriv->key_mask & BIT(keyid)) { if (keyid =3D=3D securitypriv->dot11PrivacyKeyIndex) @@ -1448,7 +1485,8 @@ u8 rtw_handle_tkip_countermeasure(struct adapter *ada= pter, const char *caller) u8 status =3D _SUCCESS; =20 if (securitypriv->btkip_countermeasure) { - unsigned long passing_ms =3D jiffies_to_msecs(jiffies - securitypriv->bt= kip_countermeasure_time); + unsigned long elapsed =3D jiffies - securitypriv->btkip_countermeasure_t= ime; + unsigned long passing_ms =3D jiffies_to_msecs(elapsed); =20 if (passing_ms > 60 * 1000) { netdev_dbg(adapter->pnetdev, --=20 2.43.0