diff options
author | delphij <delphij@FreeBSD.org> | 2014-12-15 18:28:22 +0000 |
---|---|---|
committer | delphij <delphij@FreeBSD.org> | 2014-12-15 18:28:22 +0000 |
commit | c644cc2c0ee8597464a2dffc8dac7b5ddd1bb674 (patch) | |
tree | 0ebd93784cc0900407f59223ded05412d3c42399 | |
parent | 55ed102bbcd7fb8a8c81c15f00712508cb9ea8b5 (diff) | |
download | FreeBSD-src-c644cc2c0ee8597464a2dffc8dac7b5ddd1bb674.zip FreeBSD-src-c644cc2c0ee8597464a2dffc8dac7b5ddd1bb674.tar.gz |
MFV r275784:
Plug a memory leak in libzfs. In zfs_iter_bookmarks, an nvlist is allocated
before calling lzc_get_bookmarks, which allocates the nvlist again (and
overwrites the pointer to previously allocated list).
Illumos issue:
5427 memory leak in libzfs when doing rollback
MFC after: 2 weeks
-rw-r--r-- | cddl/contrib/opensolaris/lib/libzfs/common/libzfs_iter.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_iter.c b/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_iter.c index 9698a72..aadc1f9 100644 --- a/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_iter.c +++ b/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_iter.c @@ -24,7 +24,7 @@ * Copyright (c) 2013 by Delphix. All rights reserved. * Copyright (c) 2012 Pawel Jakub Dawidek <pawel@dawidek.net>. * All rights reserved. - * Copyright 2013 Nexenta Systems, Inc. All rights reserved. + * Copyright 2014 Nexenta Systems, Inc. All rights reserved. */ #include <stdio.h> @@ -193,9 +193,6 @@ zfs_iter_bookmarks(zfs_handle_t *zhp, zfs_iter_f func, void *data) fnvlist_add_boolean(props, zfs_prop_to_name(ZFS_PROP_CREATETXG)); fnvlist_add_boolean(props, zfs_prop_to_name(ZFS_PROP_CREATION)); - /* Allocate an nvlist to hold the bookmarks. */ - bmarks = fnvlist_alloc(); - if ((err = lzc_get_bookmarks(zhp->zfs_name, props, &bmarks)) != 0) goto out; |