aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJiri Slaby <jirislaby@gmail.com>2006-11-15 00:30:17 +0100
committerLinus Torvalds <torvalds@woody.osdl.org>2006-11-14 16:02:02 -0800
commitc387fd85f84b9d89a75596325d8d6a0f730baf64 (patch)
tree3c161af64d908a29825a76acb556b7f4bd0dd4bf
parentf5ad1a785f7fb9e6f65ba437ba0a64cad4e97dae (diff)
downloadkernel_samsung_espresso10-c387fd85f84b9d89a75596325d8d6a0f730baf64.zip
kernel_samsung_espresso10-c387fd85f84b9d89a75596325d8d6a0f730baf64.tar.gz
kernel_samsung_espresso10-c387fd85f84b9d89a75596325d8d6a0f730baf64.tar.bz2
[PATCH] Char: isicom, fix close bug
port is dereferenced even if it is NULL. Dereference it _after_ the check if (!port)... Thanks Eric <ef87@yahoo.com> for reporting this. This fixes http://bugzilla.kernel.org/show_bug.cgi?id=7527 Signed-off-by: Jiri Slaby <jirislaby@gmail.com> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r--drivers/char/isicom.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/char/isicom.c b/drivers/char/isicom.c
index e9e9bf3..58c955e 100644
--- a/drivers/char/isicom.c
+++ b/drivers/char/isicom.c
@@ -1062,11 +1062,12 @@ static void isicom_shutdown_port(struct isi_port *port)
static void isicom_close(struct tty_struct *tty, struct file *filp)
{
struct isi_port *port = tty->driver_data;
- struct isi_board *card = port->card;
+ struct isi_board *card;
unsigned long flags;
if (!port)
return;
+ card = port->card;
if (isicom_paranoia_check(port, tty->name, "isicom_close"))
return;