Commit 3298a738 authored by Michael Chan's avatar Michael Chan Committed by David S. Miller
Browse files

bnx2: Fix bug in bnx2_free_rx_mem(). 



DMA memory for the jumbo rx page rings was freed incorrectly using the
wrong local variable as the array index.

Signed-off-by: default avatarMichael Chan <mchan@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 69c30e1e

drivers/net/bnx2.c

+3 −3
Original line number Diff line number Diff line
@@ -543,9 +543,9 @@ bnx2_free_rx_mem(struct bnx2 *bp) 
		for (j = 0; j < bp->rx_max_pg_ring; j++) {

			if (rxr->rx_pg_desc_ring[j])

				pci_free_consistent(bp->pdev, RXBD_RING_SIZE,

						    rxr->rx_pg_desc_ring[i],

						    rxr->rx_pg_desc_mapping[i]);

			rxr->rx_pg_desc_ring[i] = NULL;

						    rxr->rx_pg_desc_ring[j],

						    rxr->rx_pg_desc_mapping[j]);

			rxr->rx_pg_desc_ring[j] = NULL;

		}

		if (rxr->rx_pg_ring)

			vfree(rxr->rx_pg_ring);