From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olivier MATZ Subject: Re: [PATCH 5/5] test/mbuf: verify that cloning a clone works properly Date: Thu, 26 Mar 2015 09:48:24 +0100 Message-ID: <5513C7D8.6020209@6wind.com> References: <1427302838-8285-1-git-send-email-olivier.matz@6wind.com> <1427302838-8285-6-git-send-email-olivier.matz@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit To: dev-VfR2kkLFssw@public.gmane.org Return-path: In-Reply-To: <1427302838-8285-6-git-send-email-olivier.matz-pdR9zngts4EAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On 03/25/2015 06:00 PM, Olivier Matz wrote: > Signed-off-by: Olivier Matz > --- > app/test/test_mbuf.c | 26 ++++++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > [...] > > @@ -394,6 +418,8 @@ fail: > rte_pktmbuf_free(m); > if (clone) > rte_pktmbuf_free(clone); > + if (clone2) > + rte_pktmbuf_free(clone); > return -1; > } > #undef GOTO_FAIL > Self-nak on this one, it should of course be rte_pktmbuf_free(clone2)