diff options
| author | Charles Han <hanchunchao@inspur.com> | 2024-11-14 17:19:47 +0800 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.com> | 2025-02-03 22:47:32 +0100 |
| commit | 45ab5166a82d038c898985b0ad43ead69c1f9573 (patch) | |
| tree | ab8c2e3fadb190bff93a3350491d6750f6bb782a /drivers/hid/hid-winwing.c | |
| parent | 05c4ede6951b5d8e083b6bb237950cac59bdeb92 (diff) | |
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 'drivers/hid/hid-winwing.c')
| -rw-r--r-- | drivers/hid/hid-winwing.c | 2 |
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) |