Re: [U-Boot] [PATCH] sandboxfs: Fix resource leak

2016-10-25 Thread Simon Glass
On 17 October 2016 at 19:54, Simon Glass wrote: > On 17 October 2016 at 19:09, Tom Rini wrote: >> Now that we free resources in sandbox_fs_ls Coverity is letting us know >> that in some cases we might leak. So in case of error we should still >> let os_dirent_free free anything that was allocate

Re: [U-Boot] [PATCH] sandboxfs: Fix resource leak

2016-10-17 Thread Simon Glass
On 17 October 2016 at 19:09, Tom Rini wrote: > Now that we free resources in sandbox_fs_ls Coverity is letting us know > that in some cases we might leak. So in case of error we should still > let os_dirent_free free anything that was allocated. > > Fixes: 86167089b71c ("sandbox/fs: Free memory a

[U-Boot] [PATCH] sandboxfs: Fix resource leak

2016-10-17 Thread Tom Rini
Now that we free resources in sandbox_fs_ls Coverity is letting us know that in some cases we might leak. So in case of error we should still let os_dirent_free free anything that was allocated. Fixes: 86167089b71c ("sandbox/fs: Free memory allocated by os_dirent_ls") Reported-by: Coverity (CID: