diff options
| author | Amit Shah <amit.shah@redhat.com> | 2010-09-16 14:43:09 +0530 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2010-10-21 17:44:04 +1030 |
| commit | 299fb61c08c2fcd1bb6d3a4e87e53dc368475416 (patch) | |
| tree | 85a4dd7406b44c8e95c4618c5cb90ae92987cd85 | |
| parent | a461e11e7b8ca2705889bcf9582f6a8f84884bd2 (diff) | |
| download | op-kernel-dev-299fb61c08c2fcd1bb6d3a4e87e53dc368475416.zip op-kernel-dev-299fb61c08c2fcd1bb6d3a4e87e53dc368475416.tar.gz | |
virtio: console: Disable lseek(2) for port file operations
The ports are char devices; do not have seeking capabilities. Calling
nonseekable_open() from the fops_open() call and setting the llseek fops
pointer to no_llseek ensures an lseek() call from userspace returns
-ESPIPE.
Signed-off-by: Amit Shah <amit.shah@redhat.com>
CC: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
| -rw-r--r-- | drivers/char/virtio_console.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c index dc42d56..6c1b676 100644 --- a/drivers/char/virtio_console.c +++ b/drivers/char/virtio_console.c @@ -809,6 +809,8 @@ static int port_fops_open(struct inode *inode, struct file *filp) reclaim_consumed_buffers(port); spin_unlock_irq(&port->outvq_lock); + nonseekable_open(inode, filp); + /* Notify host of port being opened */ send_control_msg(filp->private_data, VIRTIO_CONSOLE_PORT_OPEN, 1); @@ -840,6 +842,7 @@ static const struct file_operations port_fops = { .poll = port_fops_poll, .release = port_fops_release, .fasync = port_fops_fasync, + .llseek = no_llseek, }; /* |
