ZFS: Clean up cppcheck warnings where relevant, leaked memory etc
authorJorgen Lundman <lundman@lundman.net>
Fri, 7 Nov 2014 01:08:35 +0000 (10:08 +0900)
committerTom Rini <trini@ti.com>
Mon, 10 Nov 2014 21:25:27 +0000 (16:25 -0500)
commite183de0d3efccd77b4a7b9d1395a9f29bba68e26
treedbae37bcf493dd637e0bcae5aa9cde4d0ac7d1f1
parent6c5f4aef02ae76357a5d85303c9890619aad0293
ZFS: Clean up cppcheck warnings where relevant, leaked memory etc

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@lundman.net>
fs/zfs/zfs.c