| From c290f8358acaeffd8e0c551ddcc24d1206143376 Mon Sep 17 00:00:00 2001 |
| From: Jiri Slaby <jslaby@suse.cz> |
| Date: Wed, 12 Oct 2011 11:32:42 +0200 |
| Subject: TTY: drop driver reference in tty_open fail path |
| |
| From: Jiri Slaby <jslaby@suse.cz> |
| |
| commit c290f8358acaeffd8e0c551ddcc24d1206143376 upstream. |
| |
| 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: 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> |
| |
| --- |
| drivers/tty/tty_io.c | 1 + |
| 1 file changed, 1 insertion(+) |
| |
| --- a/drivers/tty/tty_io.c |
| +++ b/drivers/tty/tty_io.c |
| @@ -1872,6 +1872,7 @@ got_driver: |
| if (IS_ERR(tty)) { |
| tty_unlock(); |
| mutex_unlock(&tty_mutex); |
| + tty_driver_kref_put(driver); |
| return PTR_ERR(tty); |
| } |
| } |