From: Yang Yingliang <yangyingliang@huawei.com>
Date: Fri, 28 Oct 2022 13:40:43 +0000 (+0800)
Subject: HID: hyperv: fix possible memory leak in mousevsc_probe()
X-Git-Tag: v6.1~10^2~10
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b5bcb94b0954a026bbd671741fdb00e7141f9c91;p=platform%2Fkernel%2Flinux-starfive.git

HID: hyperv: fix possible memory leak in mousevsc_probe()

If hid_add_device() returns error, it should call hid_destroy_device()
to free hid_dev which is allocated in hid_allocate_device().

Fixes: 74c4fb058083 ("HID: hv_mouse: Properly add the hid device")
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Reviewed-by: Wei Liu <wei.liu@kernel.org>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
---

diff --git a/drivers/hid/hid-hyperv.c b/drivers/hid/hid-hyperv.c
index e0bc731..ab57b49 100644
--- a/drivers/hid/hid-hyperv.c
+++ b/drivers/hid/hid-hyperv.c
@@ -499,7 +499,7 @@ static int mousevsc_probe(struct hv_device *device,
 
 	ret = hid_add_device(hid_dev);
 	if (ret)
-		goto probe_err1;
+		goto probe_err2;
 
 
 	ret = hid_parse(hid_dev);