Re: [U-Boot] [PATCH 4/8] fdtgrep: Fix logic of free() in do_fdtgrep()

Hi Heinrich, On 9 June 2018 at 13:44, Heinrich Schuchardt wrote: > On 06/09/2018 08:22 PM, Simon Glass wrote: >> This loop never actually exits, but the way the code is written this is >> not obvious. Add an explicit error check. >> >> Reported-by: Coverity (CID: 131280) >> >> Signed-off-by:

Re: [U-Boot] [PATCH 4/8] fdtgrep: Fix logic of free() in do_fdtgrep()

On 06/09/2018 08:22 PM, Simon Glass wrote: > This loop never actually exits, but the way the code is written this is > not obvious. Add an explicit error check. > > Reported-by: Coverity (CID: 131280) > > Signed-off-by: Simon Glass > --- > > tools/fdtgrep.c | 4 +++- > 1 file changed, 3

[U-Boot] [PATCH 4/8] fdtgrep: Fix logic of free() in do_fdtgrep()

This loop never actually exits, but the way the code is written this is not obvious. Add an explicit error check. Reported-by: Coverity (CID: 131280) Signed-off-by: Simon Glass --- tools/fdtgrep.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/fdtgrep.c