| Submitter | Figo.zhang |
|---|---|
| Date | June 10, 2009, 2:21 p.m. |
| Message ID | <1244643715.3422.8.camel@myhost> |
| Download | mbox | patch |
| Permalink | /patch/28411/ |
| State | Accepted |
| Delegated to: | David Miller |
| Headers | show |
Comments
From: "Figo.zhang" <figo1802@gmail.com> Date: Wed, 10 Jun 2009 22:21:55 +0800 > vfree() does its own 'NULL' check, so no need for check before > calling it. > > Signed-off-by: Figo.zhang <figo1802@gmail.com> Applied. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Patch
diff --git a/drivers/net/vxge/vxge-config.c b/drivers/net/vxge/vxge-config.c index 6b41c88..26cde57 100644 --- a/drivers/net/vxge/vxge-config.c +++ b/drivers/net/vxge/vxge-config.c @@ -1884,17 +1884,13 @@ void __vxge_hw_mempool_destroy(struct vxge_hw_mempool *mempool) mempool->memblock_size, dma_object); } - if (mempool->items_arr) - vfree(mempool->items_arr); + vfree(mempool->items_arr); - if (mempool->memblocks_dma_arr) - vfree(mempool->memblocks_dma_arr); + vfree(mempool->memblocks_dma_arr); - if (mempool->memblocks_priv_arr) - vfree(mempool->memblocks_priv_arr); + vfree(mempool->memblocks_priv_arr); - if (mempool->memblocks_arr) - vfree(mempool->memblocks_arr); + vfree(mempool->memblocks_arr); vfree(mempool); }
vfree() does its own 'NULL' check, so no need for check before calling it. Signed-off-by: Figo.zhang <figo1802@gmail.com> --- drivers/net/vxge/vxge-config.c | 12 ++++-------- 1 files changed, 4 insertions(+), 8 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html