[SRU][M/F][PATCH 1/1] Bluetooth: hci_core: Fix possible buffer overflow
Bethany Jamison
bethany.jamison at canonical.com
Wed May 1 22:15:56 UTC 2024
From: Luiz Augusto von Dentz <luiz.von.dentz at intel.com>
[ Upstream commit 81137162bfaa7278785b24c1fd2e9e74f082e8e4 ]
struct hci_dev_info has a fixed size name[8] field so in the event that
hdev->name is bigger than that strcpy would attempt to write past its
size, so this fixes this problem by switching to use strscpy.
Fixes: dcda165706b9 ("Bluetooth: hci_core: Fix build warnings")
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz at intel.com>
Signed-off-by: Sasha Levin <sashal at kernel.org>
(cherry picked from commit a41c8efe659caed0e21422876bbb6b73c15b5244 linux-6.6.y)
CVE-2024-26889
Signed-off-by: Bethany Jamison <bethany.jamison at canonical.com>
---
net/bluetooth/hci_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
index 35aaed40bfed0..f46f1e7048a37 100644
--- a/net/bluetooth/hci_core.c
+++ b/net/bluetooth/hci_core.c
@@ -908,7 +908,7 @@ int hci_get_dev_info(void __user *arg)
else
flags = hdev->flags;
- strcpy(di.name, hdev->name);
+ strscpy(di.name, hdev->name, sizeof(di.name));
di.bdaddr = hdev->bdaddr;
di.type = (hdev->bus & 0x0f) | ((hdev->dev_type & 0x03) << 4);
di.flags = flags;
--
2.34.1
More information about the kernel-team
mailing list