[dpdk-dev] [PATCH] no need to test for NULL when freeing

Thomas Monjalon thomas.monjalon at 6wind.com
Thu Jan 21 12:30:20 CET 2016


2016-01-21 12:23, David Marchand:
> free() already handles NULL pointer.
> 
> Signed-off-by: David Marchand <david.marchand at 6wind.com>
> ---
>  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?


More information about the dev mailing list