]>
Commit | Line | Data |
---|---|---|
00e5a55c BS |
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(). | |
6 | ||
7 | DMA memory for the jumbo rx page rings was freed incorrectly using the | |
8 | wrong local variable as the array index. | |
9 | ||
10 | Signed-off-by: Michael Chan <mchan@broadcom.com> | |
11 | Signed-off-by: David S. Miller <davem@davemloft.net> | |
12 | ||
13 | --- | |
14 | drivers/net/bnx2.c | 6 +++--- | |
15 | 1 file changed, 3 insertions(+), 3 deletions(-) | |
16 | ||
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; | |
29 | } | |
30 | if (rxr->rx_pg_ring) | |
31 | vfree(rxr->rx_pg_ring); |