dma: intel_mid_dma: fix double free on mid_setup_dma error path
authorAxel Lin <axel.lin@gmail.com>
Tue, 2 Nov 2010 01:52:17 +0000 (09:52 +0800)
committerDan Williams <dan.j.williams@intel.com>
Fri, 3 Dec 2010 09:46:07 +0000 (01:46 -0800)
We should not call kfree(dma) in mid_setup_dma error path because
the memory is allocated in intel_mid_dma_probe and will be freed
in intel_mid_dma_probe error path if mid_setup_dma return error.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/dma/intel_mid_dma.c

index 338bc4eed1f3f55b4de66c623a135289862c783f..4c55d451b6136a3e7990d8c35142dc87498571c2 100644 (file)
@@ -1075,7 +1075,6 @@ static int mid_setup_dma(struct pci_dev *pdev)
        if (NULL == dma->dma_pool) {
                pr_err("ERR_MDMA:pci_pool_create failed\n");
                err = -ENOMEM;
-               kfree(dma);
                goto err_dma_pool;
        }
 
@@ -1186,7 +1185,6 @@ err_engine:
        free_irq(pdev->irq, dma);
 err_irq:
        pci_pool_destroy(dma->dma_pool);
-       kfree(dma);
 err_dma_pool:
        pr_err("ERR_MDMA:setup_dma failed: %d\n", err);
        return err;