media: v4l2: Test type instead of cfg->type in v4l2_ctrl_new_custom() (07d89227) · Commits · 戴 / test

cfg->type can be overridden by v4l2_ctrl_fill() and the new value is stored in the local type var. Fix the tests to use this local var. Fixes: ("V4L/DVB: v4l2: Add new control handling framework") Cc: <stable@vger.kernel.org> Signed-off-by:

Boris Brezillon <boris.brezillon@collabora.com> [hverkuil-cisco@xs4all.nl: change to !qmenu and !qmenu_int (checkpatch)] Signed-off-by:

Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by:

Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

drivers/media/v4l2-core/v4l2-ctrls.c

+4 −5

Original line number	Diff line number	Diff line
		@@ -2464,16 +2464,15 @@ struct v4l2_ctrl v4l2_ctrl_new_custom(struct v4l2_ctrl_handler hdl,
		v4l2_ctrl_fill(cfg->id, &name, &type, &min, &max, &step,
		&def, &flags);

		is_menu = (cfg->type == V4L2_CTRL_TYPE_MENU \|\|
		cfg->type == V4L2_CTRL_TYPE_INTEGER_MENU);
		is_menu = (type == V4L2_CTRL_TYPE_MENU \|\|
		type == V4L2_CTRL_TYPE_INTEGER_MENU);
		if (is_menu)
		WARN_ON(step);
		else
		WARN_ON(cfg->menu_skip_mask);
		if (cfg->type == V4L2_CTRL_TYPE_MENU && qmenu == NULL)
		if (type == V4L2_CTRL_TYPE_MENU && !qmenu) {
		qmenu = v4l2_ctrl_get_menu(cfg->id);
		else if (cfg->type == V4L2_CTRL_TYPE_INTEGER_MENU &&
		qmenu_int == NULL) {
		} else if (type == V4L2_CTRL_TYPE_INTEGER_MENU && !qmenu_int) {
		handler_set_err(hdl, -EINVAL);
		return NULL;
		}

Admin message