diff options
author | Hans de Goede <hdegoede@redhat.com> | 2023-05-18 15:15:07 +0100 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@kernel.org> | 2023-06-09 14:57:12 +0100 |
commit | 2746a966f9f05fdb0727f4e1e8f2d51ec79e071d (patch) | |
tree | d8a3653765161eed08c977756b4c3da6f36f1bde /drivers/staging | |
parent | 76d2d7d6afae2fe5bd7f8e441e9944c6ac8f10dc (diff) | |
download | linux-stable-2746a966f9f05fdb0727f4e1e8f2d51ec79e071d.tar.gz linux-stable-2746a966f9f05fdb0727f4e1e8f2d51ec79e071d.tar.bz2 linux-stable-2746a966f9f05fdb0727f4e1e8f2d51ec79e071d.zip |
media: atomisp: gc0310: Fix double free in gc0310_remove()
gc0310_remove() must not call kfree(dev) since the gc0310_device struct
is devm managed so explicitly freeing it causes a double free.
While at it add a missing mutex_destroy() call for the input_lock.
Link: https://lore.kernel.org/r/20230518153214.194976-6-hdegoede@redhat.com
Fixes: 340b4dd6c183 ("media: atomisp: gc0310: Use devm_kzalloc() for data struct")
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/media/atomisp/i2c/atomisp-gc0310.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/media/atomisp/i2c/atomisp-gc0310.c b/drivers/staging/media/atomisp/i2c/atomisp-gc0310.c index c1590ad8f66f..af791cfc9e9a 100644 --- a/drivers/staging/media/atomisp/i2c/atomisp-gc0310.c +++ b/drivers/staging/media/atomisp/i2c/atomisp-gc0310.c @@ -377,8 +377,8 @@ static void gc0310_remove(struct i2c_client *client) v4l2_device_unregister_subdev(sd); media_entity_cleanup(&dev->sd.entity); v4l2_ctrl_handler_free(&dev->ctrls.handler); + mutex_destroy(&dev->input_lock); pm_runtime_disable(&client->dev); - kfree(dev); } static int gc0310_probe(struct i2c_client *client) |