aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/drivers/hid/hid-winwing.c
diff options
context:
space:
mode:
authorCharles Han <hanchunchao@inspur.com>2024-11-14 17:19:47 +0800
committerJiri Kosina <jkosina@suse.com>2025-02-03 22:47:32 +0100
commit45ab5166a82d038c898985b0ad43ead69c1f9573 (patch)
treeab8c2e3fadb190bff93a3350491d6750f6bb782a /drivers/hid/hid-winwing.c
parentHID: hid-steam: Fix issues with disabling both gamepad mode and lizard mode (diff)
downloadwireguard-linux-45ab5166a82d038c898985b0ad43ead69c1f9573.tar.xz
wireguard-linux-45ab5166a82d038c898985b0ad43ead69c1f9573.zip
HID: winwing: Add NULL check in winwing_init_led()
devm_kasprintf() can return a NULL pointer on failure,but this returned value in winwing_init_led() is not checked. Add NULL check in winwing_init_led(), to handle kernel NULL pointer dereference error. Fixes: 266c990debad ("HID: Add WinWing Orion2 throttle support") Signed-off-by: Charles Han <hanchunchao@inspur.com> Signed-off-by: Jiri Kosina <jkosina@suse.com>
Diffstat (limited to '')
-rw-r--r--drivers/hid/hid-winwing.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/hid/hid-winwing.c b/drivers/hid/hid-winwing.c
index 831b760c66ea..d4afbbd27807 100644
--- a/drivers/hid/hid-winwing.c
+++ b/drivers/hid/hid-winwing.c
@@ -106,6 +106,8 @@ static int winwing_init_led(struct hid_device *hdev,
"%s::%s",
dev_name(&input->dev),
info->led_name);
+ if (!led->cdev.name)
+ return -ENOMEM;
ret = devm_led_classdev_register(&hdev->dev, &led->cdev);
if (ret)