diff options
| author |   Gianluca Anzolin <gianluca@sottospazio.it> | 2013-07-25 07:26:16 +0200 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-07-26 16:34:00 -0700 |
| commit | 1d9e689c934bd5ecb0f273c6c65e0655c5cfee5f (patch) | |
| tree | 920d586f47ff1889c33dcb1f9746befb5d38098d /security | |
| parent | serial/mxs-auart: increase time to wait for transmitter to become idle (diff) | |
| download | linux-dev-1d9e689c934bd5ecb0f273c6c65e0655c5cfee5f.tar.xz linux-dev-1d9e689c934bd5ecb0f273c6c65e0655c5cfee5f.zip | |
tty_port: Fix refcounting leak in tty_port_tty_hangup()
The function tty_port_tty_hangup() could leak a reference to the tty_struct:
struct tty_struct *tty = tty_port_tty_get(port);
if (tty && (!check_clocal || !C_CLOCAL(tty))) {
tty_hangup(tty);
tty_kref_put(tty);
}
If tty != NULL and the second condition is false we never call tty_kref_put and
the reference is leaked.
Fix by always calling tty_kref_put() which accepts a NULL argument.
The patch fixes a regression introduced by commit aa27a094.
Acked-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Signed-off-by: Gianluca Anzolin <gianluca@sottospazio.it>
Acked-by: Jiri Slaby <jslaby@suse.cz>
Cc: stable <stable@vger.kernel.org> # 3.10
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'security')
0 files changed, 0 insertions, 0 deletions