summaryrefslogtreecommitdiffstats
path: root/drivers/usb
diff options
context:
space:
mode:
authorMatthias Urlichs2006-09-11 12:35:20 +0200
committerGreg Kroah-Hartman2006-09-12 12:10:20 +0200
commit5992583889ce6b7960e8023e356b2a0930ebce7c (patch)
tree487a1777c3e48dab05c546e15f9f5c9f17a07600 /drivers/usb
parentMerge branch 'audit.b28' of git://git.kernel.org/pub/scm/linux/kernel/git/vir... (diff)
downloadkernel-qcow2-linux-5992583889ce6b7960e8023e356b2a0930ebce7c.tar.gz
kernel-qcow2-linux-5992583889ce6b7960e8023e356b2a0930ebce7c.tar.xz
kernel-qcow2-linux-5992583889ce6b7960e8023e356b2a0930ebce7c.zip
usbserial: Reference leak
A sufficiently-large number of USB serial devices causes a reference leak when /proc/tty/drivers/usbserial is read. Signed-Off-By: Matthias Urlichs <smurf@smurf.noris.de> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb')
-rw-r--r--drivers/usb/serial/usb-serial.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/usb/serial/usb-serial.c b/drivers/usb/serial/usb-serial.c
index 12c1694d322e..e06a41bd0f3b 100644
--- a/drivers/usb/serial/usb-serial.c
+++ b/drivers/usb/serial/usb-serial.c
@@ -464,8 +464,10 @@ static int serial_read_proc (char *page, char **start, off_t off, int count, int
length += sprintf (page+length, " path:%s", tmp);
length += sprintf (page+length, "\n");
- if ((length + begin) > (off + count))
+ if ((length + begin) > (off + count)) {
+ usb_serial_put(serial);
goto done;
+ }
if ((length + begin) < off) {
begin += length;
length = 0;