diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2010-08-15 12:51:56 -0400 |
|---|---|---|
| committer | Arnd Bergmann <arnd@arndb.de> | 2010-09-16 04:33:16 -0400 |
| commit | d9d2e9d5c9eead1f73f92f5fc03424dab90b6c95 (patch) | |
| tree | 9e315c7eabb32658d7b03ec3454fb075bfba9c4c | |
| parent | aadbd4360911d7166dc04c5a4e5520b2c7914c73 (diff) | |
lirc: make chardev nonseekable
There does not seem to be a need for lirc to
allow seeking on the file descriptor, so let's
just disallow this before users start relying
on it.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
| -rw-r--r-- | drivers/media/IR/ir-lirc-codec.c | 1 | ||||
| -rw-r--r-- | drivers/media/IR/lirc_dev.c | 2 | ||||
| -rw-r--r-- | drivers/staging/lirc/lirc_serial.c | 1 | ||||
| -rw-r--r-- | drivers/staging/lirc/lirc_sir.c | 1 |
4 files changed, 5 insertions, 0 deletions
diff --git a/drivers/media/IR/ir-lirc-codec.c b/drivers/media/IR/ir-lirc-codec.c index 77b5946413c0..8a128934e605 100644 --- a/drivers/media/IR/ir-lirc-codec.c +++ b/drivers/media/IR/ir-lirc-codec.c | |||
| @@ -235,6 +235,7 @@ static struct file_operations lirc_fops = { | |||
| 235 | .poll = lirc_dev_fop_poll, | 235 | .poll = lirc_dev_fop_poll, |
| 236 | .open = lirc_dev_fop_open, | 236 | .open = lirc_dev_fop_open, |
| 237 | .release = lirc_dev_fop_close, | 237 | .release = lirc_dev_fop_close, |
| 238 | .llseek = no_llseek, | ||
| 238 | }; | 239 | }; |
| 239 | 240 | ||
| 240 | static int ir_lirc_register(struct input_dev *input_dev) | 241 | static int ir_lirc_register(struct input_dev *input_dev) |
diff --git a/drivers/media/IR/lirc_dev.c b/drivers/media/IR/lirc_dev.c index 899891bec352..5b145e8672fe 100644 --- a/drivers/media/IR/lirc_dev.c +++ b/drivers/media/IR/lirc_dev.c | |||
| @@ -460,6 +460,8 @@ error: | |||
| 460 | 460 | ||
| 461 | mutex_unlock(&lirc_dev_lock); | 461 | mutex_unlock(&lirc_dev_lock); |
| 462 | 462 | ||
| 463 | nonseekable_open(inode, file); | ||
| 464 | |||
| 463 | return retval; | 465 | return retval; |
| 464 | } | 466 | } |
| 465 | EXPORT_SYMBOL(lirc_dev_fop_open); | 467 | EXPORT_SYMBOL(lirc_dev_fop_open); |
diff --git a/drivers/staging/lirc/lirc_serial.c b/drivers/staging/lirc/lirc_serial.c index 9456f8e3f9ef..8da382492612 100644 --- a/drivers/staging/lirc/lirc_serial.c +++ b/drivers/staging/lirc/lirc_serial.c | |||
| @@ -1058,6 +1058,7 @@ static const struct file_operations lirc_fops = { | |||
| 1058 | .poll = lirc_dev_fop_poll, | 1058 | .poll = lirc_dev_fop_poll, |
| 1059 | .open = lirc_dev_fop_open, | 1059 | .open = lirc_dev_fop_open, |
| 1060 | .release = lirc_dev_fop_close, | 1060 | .release = lirc_dev_fop_close, |
| 1061 | .llseek = no_llseek, | ||
| 1061 | }; | 1062 | }; |
| 1062 | 1063 | ||
| 1063 | static struct lirc_driver driver = { | 1064 | static struct lirc_driver driver = { |
diff --git a/drivers/staging/lirc/lirc_sir.c b/drivers/staging/lirc/lirc_sir.c index eb08fa7138ba..2478871bd95e 100644 --- a/drivers/staging/lirc/lirc_sir.c +++ b/drivers/staging/lirc/lirc_sir.c | |||
| @@ -459,6 +459,7 @@ static const struct file_operations lirc_fops = { | |||
| 459 | .unlocked_ioctl = lirc_ioctl, | 459 | .unlocked_ioctl = lirc_ioctl, |
| 460 | .open = lirc_dev_fop_open, | 460 | .open = lirc_dev_fop_open, |
| 461 | .release = lirc_dev_fop_close, | 461 | .release = lirc_dev_fop_close, |
| 462 | .llseek = no_llseek, | ||
| 462 | }; | 463 | }; |
| 463 | 464 | ||
| 464 | static int set_use_inc(void *data) | 465 | static int set_use_inc(void *data) |
