misc: always assign miscdevice to file->private_data in open()
authorMartin Kepplinger <martink@posteo.de>
Wed, 29 Oct 2014 19:22:56 +0000 (20:22 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 7 Nov 2014 18:25:52 +0000 (10:25 -0800)
As of now, a miscdevice driver has to provide an implementation of
the open() file operation if it wants to have misc_open() assign a
pointer to struct miscdevice to file->private_data for other file
operations to use (given the user calls open()).

This leads to situations where a miscdevice driver that doesn't need
internal operations during open() has to implement open() that only
returns immediately, in order to use the data in private_data in other
fops.

This provides consistent behaviour for miscdevice developers and will
always provide the pointer in private_data. A driver's open() fop would,
of course, just overwrite it, when using private_data itself.

Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/char/misc.c

index ffa97d261cf3322615bc403931d99c71d1896a50..205ad4c584708257578d519e895b6be3a52bbe89 100644 (file)
@@ -142,8 +142,8 @@ static int misc_open(struct inode * inode, struct file * file)
 
        err = 0;
        replace_fops(file, new_fops);
+       file->private_data = c;
        if (file->f_op->open) {
-               file->private_data = c;
                err = file->f_op->open(inode,file);
        }
 fail: