Re: [PATCH] no need to test for NULL when freeing

On Thu, Jan 21, 2016 at 12:30 PM, Thomas Monjalon
[off-list ref] wrote:

2016-01-21 12:23, David Marchand:

quoted

free() already handles NULL pointer.

Signed-off-by: David Marchand <redacted>
---
 app/test/test_devargs.c                            |  3 +-
 app/test/test_link_bonding.c                       |  6 ++--
 app/test/test_pci.c                                |  3 +-
 app/test/test_ring.c                               | 36 ++++++++--------------
 drivers/net/xenvirt/rte_eth_xenvirt.c              |  6 ++--
 drivers/net/xenvirt/rte_mempool_gntalloc.c         |  9 ++----
 examples/ip_pipeline/cpu_core_map.c                |  3 +-
 .../pipeline/pipeline_flow_classification_be.c     |  6 ++--
 examples/vhost_xen/vhost_monitor.c                 |  3 +-
 examples/vhost_xen/xenstore_parse.c                | 33 +++++++-------------
 lib/librte_eal/common/eal_common_devargs.c         |  3 +-
 lib/librte_eal/linuxapp/eal/eal_memory.c           |  3 +-
 lib/librte_ether/rte_ethdev.c                      |  6 ++--
 lib/librte_kvargs/rte_kvargs.c                     |  4 +--
 14 files changed, 41 insertions(+), 83 deletions(-)

Have you used a coccinelle script?

Nop, some shell script of mine with manual checks.


-- 
David Marchand