提交 00b15628 编写于 作者: B Bojan Prtvar 提交者: Jiri Kosina

HID: prodikeys: kfree() NULL pointer cleanup

No need for freeing pm in case when it's not allocated.
Reported-by: NJulia Lawall <julia@diku.dk>
Signed-off-by: NBojan Prtvar <prtvar.b@gmail.com>
Signed-off-by: NJiri Kosina <jkosina@suse.cz>
上级 6371fe54
...@@ -816,7 +816,7 @@ static int pk_probe(struct hid_device *hdev, const struct hid_device_id *id) ...@@ -816,7 +816,7 @@ static int pk_probe(struct hid_device *hdev, const struct hid_device_id *id)
if (pm == NULL) { if (pm == NULL) {
hid_err(hdev, "can't alloc descriptor\n"); hid_err(hdev, "can't alloc descriptor\n");
ret = -ENOMEM; ret = -ENOMEM;
goto err_free; goto err_free_pk;
} }
pm->pk = pk; pm->pk = pk;
...@@ -849,10 +849,10 @@ static int pk_probe(struct hid_device *hdev, const struct hid_device_id *id) ...@@ -849,10 +849,10 @@ static int pk_probe(struct hid_device *hdev, const struct hid_device_id *id)
err_stop: err_stop:
hid_hw_stop(hdev); hid_hw_stop(hdev);
err_free: err_free:
if (pm != NULL) kfree(pm);
kfree(pm); err_free_pk:
kfree(pk); kfree(pk);
return ret; return ret;
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册