summaryrefslogtreecommitdiffstats
path: root/drivers/media
diff options
context:
space:
mode:
authorAndrew Morton <akpm@linux-foundation.org>2007-05-26 07:54:21 -0300
committerMauro Carvalho Chehab <mchehab@infradead.org>2007-06-08 08:21:12 -0300
commitffeb9ec72e18e16d0b0835d959cdf01650758638 (patch)
tree5419b16dbf2e31447f5f2ae66fa07be3427a2dec /drivers/media
parent624fc7f52b48f69fd9d1fe7a594c211762c89825 (diff)
downloadop-kernel-dev-ffeb9ec72e18e16d0b0835d959cdf01650758638.zip
op-kernel-dev-ffeb9ec72e18e16d0b0835d959cdf01650758638.tar.gz
V4L/DVB (5699): Cinergyt2: fix file release handler
If this mutex_lock_interruptible() does fail due to signal_pending() then the state of the driver will get trashed in interesting ways, because userspace cannot and will not retry the close(). Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers/media')
-rw-r--r--drivers/media/dvb/cinergyT2/cinergyT2.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/dvb/cinergyT2/cinergyT2.c b/drivers/media/dvb/cinergyT2/cinergyT2.c
index 34d7abc..6aba5b3 100644
--- a/drivers/media/dvb/cinergyT2/cinergyT2.c
+++ b/drivers/media/dvb/cinergyT2/cinergyT2.c
@@ -519,8 +519,7 @@ static int cinergyt2_release (struct inode *inode, struct file *file)
struct dvb_device *dvbdev = file->private_data;
struct cinergyt2 *cinergyt2 = dvbdev->priv;
- if (mutex_lock_interruptible(&cinergyt2->sem))
- return -ERESTARTSYS;
+ mutex_lock(&cinergyt2->sem);
if (!cinergyt2->disconnect_pending && (file->f_flags & O_ACCMODE) != O_RDONLY) {
cancel_delayed_work(&cinergyt2->query_work);
OpenPOWER on IntegriCloud