Commit 1f5ecaf9 authored by Wei Yongjun's avatar Wei Yongjun Committed by Mauro Carvalho Chehab

[media] dibusb: fix possible memory leak in dibusb_rc_query()

'buf' is malloced in dibusb_rc_query() and should be freed before
leaving from the error handling cases, otherwise it will cause
memory leak.

Fixes: ff1c1235 ("[media] dibusb: handle error code on RC query")
Signed-off-by: default avatarWei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 33e423c4
...@@ -382,9 +382,9 @@ int dibusb_rc_query(struct dvb_usb_device *d, u32 *event, int *state) ...@@ -382,9 +382,9 @@ int dibusb_rc_query(struct dvb_usb_device *d, u32 *event, int *state)
if (buf[0] != 0) if (buf[0] != 0)
deb_info("key: %*ph\n", 5, buf); deb_info("key: %*ph\n", 5, buf);
ret:
kfree(buf); kfree(buf);
ret:
return ret; return ret;
} }
EXPORT_SYMBOL(dibusb_rc_query); EXPORT_SYMBOL(dibusb_rc_query);
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment