diff options
author | Dominik Brodowski <linux@dominikbrodowski.net> | 2008-06-14 12:51:06 -0400 |
---|---|---|
committer | Dominik Brodowski <linux@dominikbrodowski.net> | 2008-06-24 09:33:41 -0400 |
commit | c1ac02280d76de7aba8a9d43638d0f7d1fd0f820 (patch) | |
tree | 5953c5979844bd06b9d818418febb657be3596dc /drivers/pcmcia/pcmcia_resource.c | |
parent | 4cf974c57633e70a8d48f9d40e41cf192c6e062c (diff) |
pcmcia: check for pointer instead of pointer address
Bug noted by Michael Buesch: checking for the pointer address is always true.
This didn't matter much, for the very first check in pcmcia_release_window()
was for the pointer pointing to something, and the return value is ignored
here. Nonetheless, fix it.
CC: Michael Buesch <mb@bu3sch.de>
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'drivers/pcmcia/pcmcia_resource.c')
-rw-r--r-- | drivers/pcmcia/pcmcia_resource.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pcmcia/pcmcia_resource.c b/drivers/pcmcia/pcmcia_resource.c index 1d128fbd1a92..c8f77b889d42 100644 --- a/drivers/pcmcia/pcmcia_resource.c +++ b/drivers/pcmcia/pcmcia_resource.c | |||
@@ -966,7 +966,7 @@ void pcmcia_disable_device(struct pcmcia_device *p_dev) { | |||
966 | pcmcia_release_configuration(p_dev); | 966 | pcmcia_release_configuration(p_dev); |
967 | pcmcia_release_io(p_dev, &p_dev->io); | 967 | pcmcia_release_io(p_dev, &p_dev->io); |
968 | pcmcia_release_irq(p_dev, &p_dev->irq); | 968 | pcmcia_release_irq(p_dev, &p_dev->irq); |
969 | if (&p_dev->win) | 969 | if (p_dev->win) |
970 | pcmcia_release_window(p_dev->win); | 970 | pcmcia_release_window(p_dev->win); |
971 | } | 971 | } |
972 | EXPORT_SYMBOL(pcmcia_disable_device); | 972 | EXPORT_SYMBOL(pcmcia_disable_device); |