aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net/qla3xxx.c
diff options
context:
space:
mode:
authorAl Viro <viro@ftp.linux.org.uk>2006-09-24 21:54:46 -0400
committerLinus Torvalds <torvalds@g5.osdl.org>2006-09-24 23:07:49 -0400
commit855fc73bf81e8f99606d6f5914c5b7c3b079d718 (patch)
tree802cd222d609efdce191ed85369d091b86ac7771 /drivers/net/qla3xxx.c
parentee111d111b69c2a216d674081107fe3b3829aa9a (diff)
[PATCH] wrong thing iounmapped (qla3xxx)
ql3xxx_probe() does ioremap and stores result in ->mem_map_registers. On failure exit it does iounmap() of the same thing. OTOH, ql3xxx_remove() does iounmap() of ->mmap_virt_base which is (a) never assigned and (b) never used other than in that iounmap() call. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/net/qla3xxx.c')
-rw-r--r--drivers/net/qla3xxx.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/qla3xxx.c b/drivers/net/qla3xxx.c
index c184cd80a9fc..157471846349 100644
--- a/drivers/net/qla3xxx.c
+++ b/drivers/net/qla3xxx.c
@@ -3508,7 +3508,7 @@ static void __devexit ql3xxx_remove(struct pci_dev *pdev)
3508 qdev->workqueue = NULL; 3508 qdev->workqueue = NULL;
3509 } 3509 }
3510 3510
3511 iounmap(qdev->mmap_virt_base); 3511 iounmap(qdev->mem_map_registers);
3512 pci_release_regions(pdev); 3512 pci_release_regions(pdev);
3513 pci_set_drvdata(pdev, NULL); 3513 pci_set_drvdata(pdev, NULL);
3514 free_netdev(ndev); 3514 free_netdev(ndev);