All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH -next] media: omap3isp: Fix missing unlock in isp_subdev_notifier_complete()
@ 2021-04-07 14:37 Wei Yongjun
  0 siblings, 0 replies; only message in thread
From: Wei Yongjun @ 2021-04-07 14:37 UTC (permalink / raw)
  To: weiyongjun1, Sakari Ailus, Laurent Pinchart, Mauro Carvalho Chehab
  Cc: linux-media, kernel-janitors, Hulk Robot

Add the missing unlock before return from function
isp_subdev_notifier_complete() in the init error
handling case.

Fixes: ba689d933361 ("media: omap3isp: Acquire graph mutex for graph traversal")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
 drivers/media/platform/omap3isp/isp.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c
index 53025c8c7531..20f59c59ff8a 100644
--- a/drivers/media/platform/omap3isp/isp.c
+++ b/drivers/media/platform/omap3isp/isp.c
@@ -2037,8 +2037,10 @@ static int isp_subdev_notifier_complete(struct v4l2_async_notifier *async)
 	mutex_lock(&isp->media_dev.graph_mutex);
 
 	ret = media_entity_enum_init(&isp->crashed, &isp->media_dev);
-	if (ret)
+	if (ret) {
+		mutex_unlock(&isp->media_dev.graph_mutex);
 		return ret;
+	}
 
 	list_for_each_entry(sd, &v4l2_dev->subdevs, list) {
 		if (sd->notifier != &isp->notifier)


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2021-04-07 14:27 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-07 14:37 [PATCH -next] media: omap3isp: Fix missing unlock in isp_subdev_notifier_complete() Wei Yongjun

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.