Commit 212a5026 authored by Eric Dumazet's avatar Eric Dumazet Committed by Jens Axboe

cciss: memory leak in cciss_init_one()

commit 22bece00
(cciss: fix regression firmware not displayed in procfs)
added a small memory leak in cciss_init_one()
Signed-off-by: default avatarEric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: default avatarJens Axboe <jens.axboe@oracle.com>
parent 723590ed
...@@ -3889,7 +3889,7 @@ static int __devinit cciss_init_one(struct pci_dev *pdev, ...@@ -3889,7 +3889,7 @@ static int __devinit cciss_init_one(struct pci_dev *pdev,
int j = 0; int j = 0;
int rc; int rc;
int dac, return_code; int dac, return_code;
InquiryData_struct *inq_buff = NULL; InquiryData_struct *inq_buff;
if (reset_devices) { if (reset_devices) {
/* Reset the controller with a PCI power-cycle */ /* Reset the controller with a PCI power-cycle */
...@@ -4029,6 +4029,7 @@ static int __devinit cciss_init_one(struct pci_dev *pdev, ...@@ -4029,6 +4029,7 @@ static int __devinit cciss_init_one(struct pci_dev *pdev,
printk(KERN_WARNING "cciss: unable to determine firmware" printk(KERN_WARNING "cciss: unable to determine firmware"
" version of controller\n"); " version of controller\n");
} }
kfree(inq_buff);
cciss_procinit(i); cciss_procinit(i);
...@@ -4045,7 +4046,6 @@ static int __devinit cciss_init_one(struct pci_dev *pdev, ...@@ -4045,7 +4046,6 @@ static int __devinit cciss_init_one(struct pci_dev *pdev,
return 1; return 1;
clean4: clean4:
kfree(inq_buff);
kfree(hba[i]->cmd_pool_bits); kfree(hba[i]->cmd_pool_bits);
if (hba[i]->cmd_pool) if (hba[i]->cmd_pool)
pci_free_consistent(hba[i]->pdev, pci_free_consistent(hba[i]->pdev,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment