drivers: atm: nicstar: Use the correct function to free some resources
authorChristophe Jaillet <christophe.jaillet@wanadoo.fr>
Sun, 17 Jul 2016 07:03:24 +0000 (09:03 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 19 Jul 2016 18:30:26 +0000 (11:30 -0700)
In 'get_scq', 'dma_alloc_coherent' has been used to allocate some
resources, so we need to free them using 'dma_free_coherent' instead
of 'kfree'.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/atm/nicstar.c

index ddc4ceb85fc560d4c0295f86e44f6a6e7457995b..700ed15c2362cfa0608900aaf49dff6a1ceef388 100644 (file)
@@ -874,7 +874,8 @@ static scq_info *get_scq(ns_dev *card, int size, u32 scd)
        scq->skb = kmalloc(sizeof(struct sk_buff *) *
                           (size / NS_SCQE_SIZE), GFP_KERNEL);
        if (!scq->skb) {
-               kfree(scq->org);
+               dma_free_coherent(&card->pcidev->dev,
+                                 2 * size, scq->org, scq->dma);
                kfree(scq);
                return NULL;
        }