1 Commit-Id: 3298a7388c00227e736d1037328788073c80c7b4
2 From: Michael Chan <mchan@broadcom.com>
3 Date: Wed, 17 Dec 2008 19:06:08 -0800
4 Acked-by: Karsten Keil <kkeil@novell.com>
5 Subject: [PATCH] bnx2: Fix bug in bnx2_free_rx_mem().
7 DMA memory for the jumbo rx page rings was freed incorrectly using the
8 wrong local variable as the array index.
10 Signed-off-by: Michael Chan <mchan@broadcom.com>
11 Signed-off-by: David S. Miller <davem@davemloft.net>
14 drivers/net/bnx2.c | 6 +++---
15 1 file changed, 3 insertions(+), 3 deletions(-)
17 --- a/drivers/net/bnx2.c
18 +++ b/drivers/net/bnx2.c
19 @@ -547,9 +547,9 @@ bnx2_free_rx_mem(struct bnx2 *bp)
20 for (j = 0; j < bp->rx_max_pg_ring; j++) {
21 if (rxr->rx_pg_desc_ring[j])
22 pci_free_consistent(bp->pdev, RXBD_RING_SIZE,
23 - rxr->rx_pg_desc_ring[i],
24 - rxr->rx_pg_desc_mapping[i]);
25 - rxr->rx_pg_desc_ring[i] = NULL;
26 + rxr->rx_pg_desc_ring[j],
27 + rxr->rx_pg_desc_mapping[j]);
28 + rxr->rx_pg_desc_ring[j] = NULL;
31 vfree(rxr->rx_pg_ring);