Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 79b185dd authored by Al Viro's avatar Al Viro
Browse files

dmxdev: get rid of pointless clearing ->f_op 



nobody else will see that struct file after return from ->release()
anyway; just leave ->f_op as is and let __fput() do that fops_put().

Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent e84f9e57

drivers/media/dvb-core/dmxdev.c

+0 −4
Original line number Diff line number Diff line
@@ -206,8 +206,6 @@ static int dvb_dvr_release(struct inode *inode, struct file *file) 
	/* TODO */

	dvbdev->users--;

	if (dvbdev->users == 1 && dmxdev->exit == 1) {

		fops_put(file->f_op);

		file->f_op = NULL;

		mutex_unlock(&dmxdev->mutex);

		wake_up(&dvbdev->wait_queue);

	} else
@@ -1120,8 +1118,6 @@ static int dvb_demux_release(struct inode *inode, struct file *file) 
	mutex_lock(&dmxdev->mutex);

	dmxdev->dvbdev->users--;

	if(dmxdev->dvbdev->users==1 && dmxdev->exit==1) {

		fops_put(file->f_op);

		file->f_op = NULL;

		mutex_unlock(&dmxdev->mutex);

		wake_up(&dmxdev->dvbdev->wait_queue);

	} else