diff options
author | Sony Chacko <sony.chacko@qlogic.com> | 2011-04-01 10:27:59 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-04-06 15:47:10 -0400 |
commit | f848d6dd10e8e27d5dd61a8ab7174a7dde3a3db5 (patch) | |
tree | 3137c44def0f18cb2afb103fb5399f2c009da9b8 /drivers/net/qlcnic/qlcnic_main.c | |
parent | 53478fef7490c90564dd328b395f238952d95c77 (diff) |
qlcnic: Memory leak fix
Fix a memory leak in error path of pci info.
Signed-off-by: Sony Chacko <sony.chacko@qlogic.com>
Signed-off-by: Anirban Chakraborty <anirban.chakraborty@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/qlcnic/qlcnic_main.c')
-rw-r--r-- | drivers/net/qlcnic/qlcnic_main.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/net/qlcnic/qlcnic_main.c b/drivers/net/qlcnic/qlcnic_main.c index d230fdde28af..de6f86681a3c 100644 --- a/drivers/net/qlcnic/qlcnic_main.c +++ b/drivers/net/qlcnic/qlcnic_main.c | |||
@@ -464,8 +464,10 @@ qlcnic_init_pci_info(struct qlcnic_adapter *adapter) | |||
464 | 464 | ||
465 | for (i = 0; i < QLCNIC_MAX_PCI_FUNC; i++) { | 465 | for (i = 0; i < QLCNIC_MAX_PCI_FUNC; i++) { |
466 | pfn = pci_info[i].id; | 466 | pfn = pci_info[i].id; |
467 | if (pfn > QLCNIC_MAX_PCI_FUNC) | 467 | if (pfn > QLCNIC_MAX_PCI_FUNC) { |
468 | return QL_STATUS_INVALID_PARAM; | 468 | ret = QL_STATUS_INVALID_PARAM; |
469 | goto err_eswitch; | ||
470 | } | ||
469 | adapter->npars[pfn].active = (u8)pci_info[i].active; | 471 | adapter->npars[pfn].active = (u8)pci_info[i].active; |
470 | adapter->npars[pfn].type = (u8)pci_info[i].type; | 472 | adapter->npars[pfn].type = (u8)pci_info[i].type; |
471 | adapter->npars[pfn].phy_port = (u8)pci_info[i].default_port; | 473 | adapter->npars[pfn].phy_port = (u8)pci_info[i].default_port; |