diff options
| author | Oliver Neukum <oliver@neukum.org> | 2008-12-15 13:12:08 +0100 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2009-01-04 01:01:43 +0100 |
| commit | b8a832b1c0a70531b4bd69a67aa0bf72f6f2dc34 (patch) | |
| tree | 61ff346a6b1381d7d2e1cba33902fe3f5fe1c91c /drivers/hid | |
| parent | 25e61613cf3ca7f6d5f89a707b20c9eed6b74455 (diff) | |
| download | op-kernel-dev-b8a832b1c0a70531b4bd69a67aa0bf72f6f2dc34.zip op-kernel-dev-b8a832b1c0a70531b4bd69a67aa0bf72f6f2dc34.tar.gz | |
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>
Diffstat (limited to 'drivers/hid')
| -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 7685ae6..96ec1ba 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) list_del(&list->node); dev = hidraw_table[minor]; - if (!dev->open--) { + if (!--dev->open) { if (list->hidraw->exist) dev->hid->ll_driver->close(dev->hid); else |
