Re: [PATCH v3 0/2] staging: rtl8723bs: refactor rtw_aes_decrypt()

From: YuChen Lin

Date: Thu Mar 19 2026 - 13:10:44 EST


On Thu, Mar 19, 2026 at 8:46 PM Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
>
>
> Thanks.
>
> Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Thanks for the Reviewed-by!

> You don't have to do this if you don't want, but unrelated to your
> patch but the last two bytes of &le_tmp64 in rtw_BIP_verify() should
> be initialized. Just set it to zero at the start of the function.
> Otherwise it's an uninitialized variable.
>
> drivers/staging/rtl8723bs/core/rtw_security.c:1308 rtw_BIP_verify() warn: not copying enough bytes for '&le_tmp64' (8 vs 6 bytes)
>
> regards,
> dan carpenter

I will send a separate patch to initialize le_tmp64 in
rtw_BIP_verify() as you suggested.

Regards,
Lin YuChen