diff options
| author | Oliver Neukum <oliver@neukum.org> | 2008-12-15 07:12:08 -0500 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2009-01-03 19:01:43 -0500 |
| commit | b8a832b1c0a70531b4bd69a67aa0bf72f6f2dc34 (patch) | |
| tree | 61ff346a6b1381d7d2e1cba33902fe3f5fe1c91c | |
| parent | 25e61613cf3ca7f6d5f89a707b20c9eed6b74455 (diff) | |
HID: fix reference count leak hidraw
The hidraw subsystem has a bug that prevents the close syscall from ever
reaching the low level driver, leading to a resource leak. Fix by replacing
postdecrement with predecrement.
Signed-off-by: Oliver Neukum <oneukum@suse.de>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
| -rw-r--r-- | drivers/hid/hidraw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hid/hidraw.c b/drivers/hid/hidraw.c index 7685ae6808c4..96ec1bacbbf3 100644 --- a/drivers/hid/hidraw.c +++ b/drivers/hid/hidraw.c | |||
| @@ -208,7 +208,7 @@ static int hidraw_release(struct inode * inode, struct file * file) | |||
| 208 | 208 | ||
| 209 | list_del(&list->node); | 209 | list_del(&list->node); |
| 210 | dev = hidraw_table[minor]; | 210 | dev = hidraw_table[minor]; |
| 211 | if (!dev->open--) { | 211 | if (!--dev->open) { |
| 212 | if (list->hidraw->exist) | 212 | if (list->hidraw->exist) |
| 213 | dev->hid->ll_driver->close(dev->hid); | 213 | dev->hid->ll_driver->close(dev->hid); |
| 214 | else | 214 | else |
