ACK: [SRU][F][PATCH v2 0/3] CVE-2021-47101

Manuel Diewald manuel.diewald at canonical.com
Mon Oct 28 13:56:29 UTC 2024


On Mon, Oct 28, 2024 at 04:41:37PM +0900, Koichiro Den wrote:
> [Impact]
> 
> asix: fix uninit-value in asix_mdio_read()
> 
> asix_read_cmd() may read less than sizeof(smsr) bytes and in this case
> smsr will be uninitialized.
> 
> Fail log:
> BUG: KMSAN: uninit-value in asix_check_host_enable drivers/net/usb/asix_common.c:82 [inline]
> BUG: KMSAN: uninit-value in asix_check_host_enable drivers/net/usb/asix_common.c:82 [inline] drivers/net/usb/asix_common.c:497
> BUG: KMSAN: uninit-value in asix_mdio_read+0x3c1/0xb00 drivers/net/usb/asix_common.c:497 drivers/net/usb/asix_common.c:497
>  asix_check_host_enable drivers/net/usb/asix_common.c:82 [inline]
>  asix_check_host_enable drivers/net/usb/asix_common.c:82 [inline] drivers/net/usb/asix_common.c:497
>  asix_mdio_read+0x3c1/0xb00 drivers/net/usb/asix_common.c:497 drivers/net/usb/asix_common.c:497
> 
> [Fix]
> 
> Noble:  not affected
> Jammy:  fixed via stable
> Focal:  Clean cherry-pick following two prereq commits backporting
> Bionic: fix sent to esm ML
> Xenial: not affected
> Trusty: not affected
> 
> [Test Case]
> 
> Compile tested / Smatch tested on the changed file (with amd64 generic config) [*]
> 
> [*]: warn message found, which is irrelevant to the CVE backport.
>      $ kchecker drivers/net/usb/asix_common.c
>        --(snip)--
>        CHECK   drivers/net/usb/asix_common.c
>      drivers/net/usb/asix_common.c:634 asix_get_eeprom() warn: potential spectre issue 'eeprom_buff' [w]
> 
> [Where problems could occur]
> 
> This backport affects those who use ASIX USB Ethernet devices, an issue
> with it would be visible to the user via unpredicted system behavior or
> a system crash especially if some sort of regression will be found for
> the prerequisite fix commit in the future.
> 
> [Notes]
> 
> v2:
>   - Pull a follow-up fix commit from upstream that fixes the first prerequisite
>     commit "net: asix: fix uninit value bugs". Also, fix [Impact] section in this cover letter.
> 
> Pavel Skripkin (3):
>   net: asix: fix uninit value bugs
>   asix: fix wrong return value in asix_check_host_enable()
>   asix: fix uninit-value in asix_mdio_read()
> 
>  drivers/net/usb/asix_common.c | 73 ++++++++++++++++-------------------
>  1 file changed, 33 insertions(+), 40 deletions(-)
> 
> -- 
> 2.43.0
> 
> 
> -- 
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Acked-by: Manuel Diewald <manuel.diewald at canonical.com>

-- 
 Manuel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20241028/6e15b6c5/attachment.sig>


More information about the kernel-team mailing list