lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

bnx2: no need to check before vfreeing

Browse Source 
There is no need to check if a pointer is NULL before calling
vfree(), since vfree() function already check for it.

Signed-off-by: Breno Leitão <leitao@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Breno Leitao
2009-06-08 10:30:19 +00:00
committed by David S. Miller
parent 3d8fd38567
commit 25b0b999fb
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/net/bnx2.c
View File

@@ -546,8 +546,7 @@ bnx2_free_rx_mem(struct bnx2 *bp)
rxr->rx_desc_mapping[j]); rxr->rx_desc_mapping[j]);
rxr->rx_desc_ring[j] = NULL; rxr->rx_desc_ring[j] = NULL;
} }
if (rxr->rx_buf_ring) vfree(rxr->rx_buf_ring);
vfree(rxr->rx_buf_ring);
rxr->rx_buf_ring = NULL; rxr->rx_buf_ring = NULL;
for (j = 0; j < bp->rx_max_pg_ring; j++) { for (j = 0; j < bp->rx_max_pg_ring; j++) {
@@ -557,8 +556,7 @@ bnx2_free_rx_mem(struct bnx2 *bp)
rxr->rx_pg_desc_mapping[j]); rxr->rx_pg_desc_mapping[j]);
rxr->rx_pg_desc_ring[j] = NULL; rxr->rx_pg_desc_ring[j] = NULL;
} }
if (rxr->rx_pg_ring) vfree(rxr->rx_pg_ring);
vfree(rxr->rx_pg_ring);
rxr->rx_pg_ring = NULL; rxr->rx_pg_ring = NULL;
} }
} }