From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v4 3/6] test: fix memory leak in ring autotest Date: Tue, 06 Feb 2018 01:36:20 +0100 Message-ID: <5821346.polxSJYkpG@xps> References: <409be7970a029d2d2cfffeafee43ea2645244eb2.1517834482.git.anatoly.burakov@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Olivier Matz , pablo.de.lara.guarch@intel.com, stable@dpdk.org To: Anatoly Burakov Return-path: In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 05/02/2018 14:03, Anatoly Burakov: > Get rid of global static ring variable and don't reuse rings > between test runs. > > Fixes: 4e32101f9b01 ("ring: support freeing") > Cc: pablo.de.lara.guarch@intel.com > Cc: stable@dpdk.org > > Signed-off-by: Anatoly Burakov > Acked-by: Olivier Matz There is an error caught by clang: test/test/test_ring.c:676:6: fatal error: variable 'rp' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]