media: exynos4-is: Fix a reference count leak due to pm_runtime_get_sync
authorQiushi Wu <wu000273@umn.edu>
Sun, 14 Jun 2020 03:10:58 +0000 (05:10 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 29 Oct 2020 08:05:42 +0000 (09:05 +0100)
[ Upstream commit c47f7c779ef0458a58583f00c9ed71b7f5a4d0a2 ]

On calling pm_runtime_get_sync() the reference count of the device
is incremented. In case of failure, decrement the
reference count before returning the error.

Signed-off-by: Qiushi Wu <wu000273@umn.edu>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/media/platform/exynos4-is/media-dev.c

index 532b7cd361dc8a08b45964f71009ced9ff132e30..a1599659b88ba680c7322fc829953a4e476f0042 100644 (file)
@@ -477,8 +477,10 @@ static int fimc_md_register_sensor_entities(struct fimc_md *fmd)
                return -ENXIO;
 
        ret = pm_runtime_get_sync(fmd->pmf);
-       if (ret < 0)
+       if (ret < 0) {
+               pm_runtime_put(fmd->pmf);
                return ret;
+       }
 
        fmd->num_sensors = 0;