diff options
author | Amit Kumar Salecha <amit.salecha@qlogic.com> | 2010-07-18 17:51:59 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-07-18 17:51:59 -0400 |
commit | bfc978fa5f3005e5dfb39c52393c3339f4f00233 (patch) | |
tree | 8993955e9e4c6156696811800ec6d44536e417c5 /drivers | |
parent | 4a1745fc54e22e9fa928d72f97ee0e91449c9fd0 (diff) |
qlcnic: fix pci resource leak
pci_get_domain_bus_and_slot: caller must decrement the
reference count by calling pci_dev_put().
Signed-off-by: Amit Kumar Salecha <amit.salecha@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/qlcnic/qlcnic_main.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/net/qlcnic/qlcnic_main.c b/drivers/net/qlcnic/qlcnic_main.c index 8d2d62ff1a37..f1f7acfbf412 100644 --- a/drivers/net/qlcnic/qlcnic_main.c +++ b/drivers/net/qlcnic/qlcnic_main.c | |||
@@ -2695,9 +2695,14 @@ static int qlcnic_is_first_func(struct pci_dev *pdev) | |||
2695 | oth_pdev = pci_get_domain_bus_and_slot(pci_domain_nr | 2695 | oth_pdev = pci_get_domain_bus_and_slot(pci_domain_nr |
2696 | (pdev->bus), pdev->bus->number, | 2696 | (pdev->bus), pdev->bus->number, |
2697 | PCI_DEVFN(PCI_SLOT(pdev->devfn), val)); | 2697 | PCI_DEVFN(PCI_SLOT(pdev->devfn), val)); |
2698 | if (!oth_pdev) | ||
2699 | continue; | ||
2698 | 2700 | ||
2699 | if (oth_pdev && (oth_pdev->current_state != PCI_D3cold)) | 2701 | if (oth_pdev->current_state != PCI_D3cold) { |
2702 | pci_dev_put(oth_pdev); | ||
2700 | return 0; | 2703 | return 0; |
2704 | } | ||
2705 | pci_dev_put(oth_pdev); | ||
2701 | } | 2706 | } |
2702 | return 1; | 2707 | return 1; |
2703 | } | 2708 | } |