linux-kernel - Re: [patch] video: fix build bugs in drivers/media/video/stk-webcam.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20080509171012.GA3473@singular.sob>
Date:	Fri, 9 May 2008 18:10:12 +0100
From:	Jaime Velasco Juan <jsagarribay@...il.com>
To:	Mauro Carvalho Chehab <mchehab@...radead.org>
Cc:	Ingo Molnar <mingo@...e.hu>, linux-kernel@...r.kernel.org
Subject: Re: [patch] video: fix build bugs in
	drivers/media/video/stk-webcam.c

El mié. 07 de may. de 2008, a las 16:29:50 -0300, Mauro Carvalho Chehab escribió:
> The V4L1 symbols are used to help us to track what drivers are still using the
> depreciated V4L1 API, or otherwise provide support for V4L1 compatibility
> layer. This is not the case of stkwebcam.
> 
> So, instead of the proposed patch, I prefer just to disable the additional
> non-V4L2 sysfs interface, if the flag is not enabled.

Yeah, that might be the best solution to this problem.

> 
> There are some discussions at the ML to create a standard to allow configuring
> V4L2 controls via sysfs, but this is not standard yet. The idea is to add some
> support at videodev.h using the same handlers defined by video_ioctl2.
> Unfortunately, people stopped discussing this, due to the lack of time and
> people to work on this.
> 
> I think this could be a nice addition to V4L2, and will help to avoid creating
> non-standardized interfaces at sysfs.
> 
> > For the record, this bug was found in march, but it seems
> > I forgot to CC: the patch to Mauro, Sorry about that.
> > 
> > http://marc.info/?l=linux-video&m=120594964524615&w=2
> 
> I probably missed this.
> 
> ---
> 
> Instead of Ingo's approach, this seems to be cleaner:
> 
> Fix compilation, when V4L1_COMPAT is disabled
> 
> From: Mauro Carvalho Chehab <mchehab@...radead.org>
> 
> This driver uses some sysfs helper functions that are available only for legacy
> drivers. It also requires linux/mm.h.
> 
> This patch fixes compiliation when not in compat mode.
> 
> Thanks to Ingo Molnar for identifying this issue.
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab@...radead.org>
Acked-by: Jaime Velasco Juan <jsagarribay@...il.com>

Regards

> ---
> 
>  drivers/media/video/stk-webcam.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> 
> diff --git a/drivers/media/video/stk-webcam.c b/drivers/media/video/stk-webcam.c
> index 9276ed9..b12c60c 100644
> --- a/drivers/media/video/stk-webcam.c
> +++ b/drivers/media/video/stk-webcam.c
> @@ -30,6 +30,7 @@
>  #include <linux/kref.h>
>  
>  #include <linux/usb.h>
> +#include <linux/mm.h>
>  #include <linux/vmalloc.h>
>  #include <linux/videodev2.h>
>  #include <media/v4l2-common.h>
> @@ -245,6 +246,8 @@ static int stk_initialise(struct stk_camera *dev)
>  		return -1;
>  }
>  
> +#ifdef CONFIG_VIDEO_V4L1_COMPAT
> +
>  /* sysfs functions */
>  /*FIXME cleanup this */
>  
> @@ -350,6 +353,10 @@ static void stk_remove_sysfs_files(struct video_device *vdev)
>  	video_device_remove_file(vdev, &dev_attr_vflip);
>  }
>  
> +#else
> +#define stk_create_sysfs_files(a)
> +#define stk_remove_sysfs_files(a)
> +#endif
>  
>  /* *********************************************** */
>  /*
> 
> 
> 
> 
> 
> Cheers,
> Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/