]> www.infradead.org Git - users/jedix/linux-maple.git/commit
iommu/amd: Fix potential buffer overflow in parse_ivrs_acpihid
authorPavel Paklov <Pavel.Paklov@cyberprotect.ru>
Tue, 25 Mar 2025 09:22:44 +0000 (09:22 +0000)
committerJoerg Roedel <jroedel@suse.de>
Thu, 17 Apr 2025 14:37:21 +0000 (16:37 +0200)
commit8dee308e4c01dea48fc104d37f92d5b58c50b96c
tree8ed706299799afd0e4e4a5c44cacf8ef01ff5c8d
parent2d00c34d665bc23f5200962dbc4ac1919317036c
iommu/amd: Fix potential buffer overflow in parse_ivrs_acpihid

There is a string parsing logic error which can lead to an overflow of hid
or uid buffers. Comparing ACPIID_LEN against a total string length doesn't
take into account the lengths of individual hid and uid buffers so the
check is insufficient in some cases. For example if the length of hid
string is 4 and the length of the uid string is 260, the length of str
will be equal to ACPIID_LEN + 1 but uid string will overflow uid buffer
which size is 256.

The same applies to the hid string with length 13 and uid string with
length 250.

Check the length of hid and uid strings separately to prevent
buffer overflow.

Found by Linux Verification Center (linuxtesting.org) with SVACE.

Fixes: ca3bf5d47cec ("iommu/amd: Introduces ivrs_acpihid kernel parameter")
Cc: stable@vger.kernel.org
Signed-off-by: Pavel Paklov <Pavel.Paklov@cyberprotect.ru>
Link: https://lore.kernel.org/r/20250325092259.392844-1-Pavel.Paklov@cyberprotect.ru
Signed-off-by: Joerg Roedel <jroedel@suse.de>
drivers/iommu/amd/init.c