qlge: bugfix: Add missing dev_kfree_skb_any() call.
authorRon Mercer <ron.mercer@qlogic.com>
Fri, 13 Feb 2009 00:37:48 +0000 (16:37 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 13 Feb 2009 00:37:48 +0000 (16:37 -0800)
We put the skb back if we can't get mapping for it.  We don't
want unmapped buffers on our receive buffer queue.

Signed-off-by: Ron Mercer <ron.mercer@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/qlge/qlge_main.c

index 5b75fc9..fc7d210 100644 (file)
@@ -970,6 +970,8 @@ static void ql_update_sbq(struct ql_adapter *qdev, struct rx_ring *rx_ring)
                                if (pci_dma_mapping_error(qdev->pdev, map)) {
                                        QPRINTK(qdev, IFUP, ERR, "PCI mapping failed.\n");
                                        rx_ring->sbq_clean_idx = clean_idx;
+                                       dev_kfree_skb_any(sbq_desc->p.skb);
+                                       sbq_desc->p.skb = NULL;
                                        return;
                                }
                                pci_unmap_addr_set(sbq_desc, mapaddr, map);