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

[media] rc: nuvoton-cir: fix potential double free in nvt_probe()

Since rc_unregister_device() frees its argument, the subsequently
call to rc_free_device() on the same variable will cause a double
free bug. Fix by set argument to NULL, thus when fall through to
rc_free_device(), nothing will be done there.
Signed-off-by: default avatarWei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 408208fb
...@@ -1107,6 +1107,7 @@ static int nvt_probe(struct pnp_dev *pdev, const struct pnp_device_id *dev_id) ...@@ -1107,6 +1107,7 @@ static int nvt_probe(struct pnp_dev *pdev, const struct pnp_device_id *dev_id)
release_region(nvt->cir_addr, CIR_IOREG_LENGTH); release_region(nvt->cir_addr, CIR_IOREG_LENGTH);
exit_unregister_device: exit_unregister_device:
rc_unregister_device(rdev); rc_unregister_device(rdev);
rdev = NULL;
exit_free_dev_rdev: exit_free_dev_rdev:
rc_free_device(rdev); rc_free_device(rdev);
kfree(nvt); kfree(nvt);
......
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