Re: [U-Boot] [PATCH 1/2] fdtdec: test: Fix memory leak

2019-06-28 Thread Simon Glass
On Tue, 21 May 2019 at 10:43, Simon Glass wrote: > > Hi Thierry, > > On Tue, 21 May 2019 at 04:21, Thierry Reding wrote: > > > > On Mon, May 20, 2019 at 02:51:08PM -0600, Simon Glass wrote: > > > Hi Thierry, > > > > > > On Mon, 20 May 2019 at 10:05, Thierry Reding > > > wrote: > > > > > > > >

Re: [U-Boot] [PATCH 1/2] fdtdec: test: Fix memory leak

2019-05-21 Thread Simon Glass
Hi Thierry, On Tue, 21 May 2019 at 04:21, Thierry Reding wrote: > > On Mon, May 20, 2019 at 02:51:08PM -0600, Simon Glass wrote: > > Hi Thierry, > > > > On Mon, 20 May 2019 at 10:05, Thierry Reding > > wrote: > > > > > > From: Thierry Reding > > > > > > Free the memory allocated to store the

Re: [U-Boot] [PATCH 1/2] fdtdec: test: Fix memory leak

2019-05-21 Thread Thierry Reding
On Mon, May 20, 2019 at 02:51:08PM -0600, Simon Glass wrote: > Hi Thierry, > > On Mon, 20 May 2019 at 10:05, Thierry Reding wrote: > > > > From: Thierry Reding > > > > Free the memory allocated to store the test FDT upon test completion to > > avoid leaking the memory. We don't bother cleaning

Re: [U-Boot] [PATCH 1/2] fdtdec: test: Fix memory leak

2019-05-20 Thread Simon Glass
Hi Thierry, On Mon, 20 May 2019 at 10:05, Thierry Reding wrote: > > From: Thierry Reding > > Free the memory allocated to store the test FDT upon test completion to > avoid leaking the memory. We don't bother cleaning up on test failure > since the code is broken in that case and should be

[U-Boot] [PATCH 1/2] fdtdec: test: Fix memory leak

2019-05-20 Thread Thierry Reding
From: Thierry Reding Free the memory allocated to store the test FDT upon test completion to avoid leaking the memory. We don't bother cleaning up on test failure since the code is broken in that case and should be fixed, in which case the leak would also go away. Reported-by: Tom Rini