mirror of
https://github.com/Fishwaldo/Star64_linux.git
synced 2025-06-28 09:31:14 +00:00
[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>
This commit is contained in:
parent
ee111d111b
commit
855fc73bf8
1 changed files with 1 additions and 1 deletions
|
@ -3508,7 +3508,7 @@ static void __devexit ql3xxx_remove(struct pci_dev *pdev)
|
||||||
qdev->workqueue = NULL;
|
qdev->workqueue = NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
iounmap(qdev->mmap_virt_base);
|
iounmap(qdev->mem_map_registers);
|
||||||
pci_release_regions(pdev);
|
pci_release_regions(pdev);
|
||||||
pci_set_drvdata(pdev, NULL);
|
pci_set_drvdata(pdev, NULL);
|
||||||
free_netdev(ndev);
|
free_netdev(ndev);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue