From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] no need to test for NULL when freeing Date: Thu, 21 Jan 2016 12:30:20 +0100 Message-ID: <3415689.rt0oTMTjK5@xps13> References: <1453375399-25576-1-git-send-email-david.marchand@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: David Marchand Return-path: Received: from mail-wm0-f45.google.com (mail-wm0-f45.google.com [74.125.82.45]) by dpdk.org (Postfix) with ESMTP id 78AD28E7B for ; Thu, 21 Jan 2016 12:31:21 +0100 (CET) Received: by mail-wm0-f45.google.com with SMTP id l65so215974199wmf.1 for ; Thu, 21 Jan 2016 03:31:21 -0800 (PST) In-Reply-To: <1453375399-25576-1-git-send-email-david.marchand@6wind.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-01-21 12:23, David Marchand: > free() already handles NULL pointer. > > Signed-off-by: David Marchand > --- > 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?