From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.droids-corp.org (zoll.droids-corp.org [94.23.50.67]) by dpdk.org (Postfix) with ESMTP id 986CDCE7 for ; Fri, 27 Feb 2015 14:29:00 +0100 (CET) Received: from was59-1-82-226-113-214.fbx.proxad.net ([82.226.113.214] helo=[192.168.0.10]) by mail.droids-corp.org with esmtpsa (TLS1.2:DHE_RSA_AES_128_CBC_SHA1:128) (Exim 4.80) (envelope-from ) id 1YRL2D-0004Iz-Rd; Fri, 27 Feb 2015 14:32:57 +0100 Message-ID: <54F07113.5030200@6wind.com> Date: Fri, 27 Feb 2015 14:28:51 +0100 From: Olivier MATZ User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.3.0 MIME-Version: 1.0 To: Pawel Wodkowski , dev@dpdk.org References: <1424700600-1765-1-git-send-email-pawelx.wodkowski@intel.com> <1424868086-4232-1-git-send-email-pawelx.wodkowski@intel.com> In-Reply-To: <1424868086-4232-1-git-send-email-pawelx.wodkowski@intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2 0/4] Fix issues reported by static analysis tool X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 27 Feb 2015 13:29:00 -0000 Hi Pawel, On 02/25/2015 01:41 PM, Pawel Wodkowski wrote: > Static analysis report some issues against current DPDK version. Most of > them need only cosmetic code changes (changing type of variable). > > One issue related with ring pmd fix real memory leak problem. > > PATCH v2 changes: > - remove patch 5/5 as it was NACKed > - reword commit log acording to mailing list sugestions. > > Pawel Wodkowski (4): > rte_timer: change declaration of rte_timer_cb_t > librte_kvargs: make rte_kvargs_free() be consistent with other > "free()" functions > pmd ring: fix possible memory leak during devinit > cmdline: make parse_set_list() use size_t instead of int for low/high > parameter > > lib/librte_cmdline/cmdline_parse_portlist.c | 4 ++-- > lib/librte_kvargs/rte_kvargs.c | 4 ++++ > lib/librte_kvargs/rte_kvargs.h | 3 ++- > lib/librte_pmd_ring/rte_eth_ring.c | 6 +++--- > lib/librte_timer/rte_timer.h | 4 ++-- > 5 files changed, 13 insertions(+), 8 deletions(-) > Series: Acked-by: Olivier Matz