diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-05-09 14:59:16 +0300 |
---|---|---|
committer | Ben Hutchings <ben@decadent.org.uk> | 2014-08-06 18:07:41 +0100 |
commit | 12489e9cb91ea91513cb991dadd1b1b9c9e4cc3e (patch) | |
tree | ac68da8193af25d2311f974b2b3b21523a622cfa | |
parent | 6806fa8b6795aba9be8742a8f598f60eed26f875 (diff) | |
download | lwn-12489e9cb91ea91513cb991dadd1b1b9c9e4cc3e.tar.gz lwn-12489e9cb91ea91513cb991dadd1b1b9c9e4cc3e.zip |
applicom: dereferencing NULL on error path
commit 8bab797c6e5724a43b7666ad70860712365cdb71 upstream.
This is a static checker fix. The "dev" variable is always NULL after
the while statement so we would be dereferencing a NULL pointer here.
Fixes: 819a3eba4233 ('[PATCH] applicom: fix error handling')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
-rw-r--r-- | drivers/char/applicom.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/char/applicom.c b/drivers/char/applicom.c index 25373df1dcf8..5d069c79bd8b 100644 --- a/drivers/char/applicom.c +++ b/drivers/char/applicom.c @@ -345,7 +345,6 @@ out: free_irq(apbs[i].irq, &dummy); iounmap(apbs[i].RamIO); } - pci_disable_device(dev); return ret; } |