[U-Boot] [U-Boot, v1] ZFS: Clean up cppcheck warnings where relevant, leaked memory etc

Tom Rini trini at ti.com
Mon Nov 10 22:28:36 CET 2014


On Fri, Nov 07, 2014 at 10:08:35AM +0900, Jorgen Lundman wrote:

> In a message from Wolfgang Denk highlighting warnings from cppcheck,
> the patch will address those that are correctly diagnosed. Some are
> false-positives:
> 
> > [fs/zfs/zfs.c:937]: (error) Memory leak: l
> dmu_read() allocates "l" if successful, so error-case should not free
> it.
> > [fs/zfs/zfs.c:1141]: (error) Memory leak: dnbuf
> dmu_read() allocates "dnbuf" if successful, so error-case should not
> free it.
> > [fs/zfs/zfs.c:1372]: (error) Memory leak: osp
> zio_read() allocates "osp" if successful, so error-case should
> not free it.
> > [fs/zfs/zfs.c:1726]: (error) Memory leak: nvlist
> int_zfs_fetch_nvlist() allocates "nvlist" if successful, so error-case
> should not free it.
> 
> Signed-off-by: Jorgen Lundman <lundman at lundman.net>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141110/c0e64719/attachment.pgp>


More information about the U-Boot mailing list