diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-09-24 01:18:45 -0400 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2013-10-24 08:11:16 -0400 |
commit | de524681f88ff4ed293aa239f83c8cb04d59b47d (patch) | |
tree | c533a3ec17acdd2213545792b77f88f675fbe4cf | |
parent | 3de9e65f011b95235a789b12abc4730570cdb737 (diff) |
igbvf: add missing iounmap() on error in igbvf_probe()
Add the missing iounmap() before return from igbvf_probe()
in the error handling case.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Tested-by: Sibai Li <Sibai.li@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
-rw-r--r-- | drivers/net/ethernet/intel/igbvf/netdev.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/igbvf/netdev.c b/drivers/net/ethernet/intel/igbvf/netdev.c index f48ae71f0d47..9fadbb28cf08 100644 --- a/drivers/net/ethernet/intel/igbvf/netdev.c +++ b/drivers/net/ethernet/intel/igbvf/netdev.c | |||
@@ -2698,7 +2698,7 @@ static int igbvf_probe(struct pci_dev *pdev, const struct pci_device_id *ent) | |||
2698 | if (ei->get_variants) { | 2698 | if (ei->get_variants) { |
2699 | err = ei->get_variants(adapter); | 2699 | err = ei->get_variants(adapter); |
2700 | if (err) | 2700 | if (err) |
2701 | goto err_ioremap; | 2701 | goto err_get_variants; |
2702 | } | 2702 | } |
2703 | 2703 | ||
2704 | /* setup adapter struct */ | 2704 | /* setup adapter struct */ |
@@ -2795,6 +2795,7 @@ err_hw_init: | |||
2795 | kfree(adapter->rx_ring); | 2795 | kfree(adapter->rx_ring); |
2796 | err_sw_init: | 2796 | err_sw_init: |
2797 | igbvf_reset_interrupt_capability(adapter); | 2797 | igbvf_reset_interrupt_capability(adapter); |
2798 | err_get_variants: | ||
2798 | iounmap(adapter->hw.hw_addr); | 2799 | iounmap(adapter->hw.hw_addr); |
2799 | err_ioremap: | 2800 | err_ioremap: |
2800 | free_netdev(netdev); | 2801 | free_netdev(netdev); |