diff options
author | Jiri Slaby <jslaby@suse.cz> | 2011-10-12 05:32:42 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-10-18 17:22:36 -0400 |
commit | c290f8358acaeffd8e0c551ddcc24d1206143376 (patch) | |
tree | f28068fd3da69c77b179d3955ec678c7ddeb7067 /drivers/tty/tty_io.c | |
parent | 64d91cfaade2155ad048fe3b65238a052e29dde4 (diff) |
TTY: drop driver reference in tty_open fail path
When tty_driver_lookup_tty fails in tty_open, we forget to drop a
reference to the tty driver. This was added by commit 4a2b5fddd5 (Move
tty lookup/reopen to caller).
Fix that by adding tty_driver_kref_put to the fail path.
I will refactor the code later. This is for the ease of backporting to
stable.
Introduced-in: v2.6.28-rc2
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Cc: stable <stable@vger.kernel.org>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Acked-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/tty/tty_io.c')
-rw-r--r-- | drivers/tty/tty_io.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/tty/tty_io.c b/drivers/tty/tty_io.c index 4ca4bcd28ff7..6913da8f202c 100644 --- a/drivers/tty/tty_io.c +++ b/drivers/tty/tty_io.c | |||
@@ -1873,6 +1873,7 @@ got_driver: | |||
1873 | if (IS_ERR(tty)) { | 1873 | if (IS_ERR(tty)) { |
1874 | tty_unlock(); | 1874 | tty_unlock(); |
1875 | mutex_unlock(&tty_mutex); | 1875 | mutex_unlock(&tty_mutex); |
1876 | tty_driver_kref_put(driver); | ||
1876 | return PTR_ERR(tty); | 1877 | return PTR_ERR(tty); |
1877 | } | 1878 | } |
1878 | } | 1879 | } |