[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <d315073f004ce46e0198fd614398e046ffe649e7.1487111824.git.pavel@ucw.cz>
Date: Tue, 14 Feb 2017 23:38:49 +0100
From: Pavel Machek <pavel@....cz>
To: sakari.ailus@....fi
Cc: sre@...nel.org, pali.rohar@...il.com, pavel@....cz,
linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
laurent.pinchart@...asonboard.com, mchehab@...nel.org,
ivo.g.dimitrov.75@...il.com
Subject: [PATCH 1/4] v4l2: device_register_subdev_nodes: allow calling
multiple times
From: Sebastian Reichel <sre@...nel.org>
If v4l2_device_register_subdev_nodes() is called multiple times, it is
better to return early than corrupt memory.
Without this, exposure / gain controls do not work in the camera
application on N900.
Signed-off-by: Sebastian Reichel <sre@...nel.org>
Signed-off-by: Ivaylo Dimitrov <ivo.g.dimitrov.75@...il.com>
Signed-off-by: Pavel Machek <pavel@....cz>
---
drivers/media/v4l2-core/v4l2-device.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/v4l2-core/v4l2-device.c b/drivers/media/v4l2-core/v4l2-device.c
index f364cc1..937c6de 100644
--- a/drivers/media/v4l2-core/v4l2-device.c
+++ b/drivers/media/v4l2-core/v4l2-device.c
@@ -235,6 +235,9 @@ int v4l2_device_register_subdev_nodes(struct v4l2_device *v4l2_dev)
if (!(sd->flags & V4L2_SUBDEV_FL_HAS_DEVNODE))
continue;
+ if (sd->devnode)
+ continue;
+
vdev = kzalloc(sizeof(*vdev), GFP_KERNEL);
if (!vdev) {
err = -ENOMEM;
--
2.1.4
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Download attachment "signature.asc" of type "application/pgp-signature" (182 bytes)
Powered by blists - more mailing lists