]> git.baikalelectronics.ru Git - kernel.git/commit
[media] cx25821: do not expose broken video output streams
authorHans Verkuil <hans.verkuil@cisco.com>
Sun, 14 Apr 2013 13:53:07 +0000 (10:53 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 15 Apr 2013 11:28:41 +0000 (08:28 -0300)
commit05b2b1b339d4ee03c8a018e5e2461a525bea1f5f
treeb3f9fc3edd2608e1cdb4313ed89a4fd17f4481c0
parent7e9dad6bc63d0f2a37325387720c3c01077d64c1
[media] cx25821: do not expose broken video output streams

The cx25821 driver has support for one audio output channel and two video
output channels.
This is implemented in a very ugly and very evil way through a custom ioctl
that passes the filename of a file containing the video data, which is then
read by the driver itself using vfs.
There are a number of problems with this:
1) it's very ugly and very evil (I can't say that often enough).
2) V4L2 supports video output, so why not use that?
3) it's very buggy, closing the filehandle through which you passed the ioctl
   will oops the kernel.
4) it's a nasty security leak since this allows you to load any file in the
   system as a video or audio source, so in theory you can output /etc/passwd
   to audio or video out and record & decode it on another device.
Because of all these issues we no longer register those output video nodes.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/pci/cx25821/cx25821-video.c