Author: delphij
Date: Fri Mar 14 01:10:54 2014
New Revision: 263128
URL: http://svnweb.freebsd.org/changeset/base/263128

Log:
  MFC r262577: MFV r262570:
  
  4626 libzfs memleak in zpool_in_use()

Modified:
  stable/9/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_import.c
Directory Properties:
  stable/9/cddl/contrib/opensolaris/   (props changed)
  stable/9/cddl/contrib/opensolaris/lib/libzfs/   (props changed)

Changes in other areas also in this revision:
Modified:
  stable/8/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_import.c
Directory Properties:
  stable/8/cddl/contrib/opensolaris/   (props changed)
  stable/8/cddl/contrib/opensolaris/lib/libzfs/   (props changed)

Modified: stable/9/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_import.c
==============================================================================
--- stable/9/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_import.c Fri Mar 
14 01:09:42 2014        (r263127)
+++ stable/9/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_import.c Fri Mar 
14 01:10:54 2014        (r263128)
@@ -20,8 +20,8 @@
  */
 /*
  * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
- * Copyright 2011 Nexenta Systems, Inc. All rights reserved.
  * Copyright (c) 2012 by Delphix. All rights reserved.
+ * Copyright 2014 Nexenta Systems, Inc. All rights reserved.
  */
 
 /*
@@ -1613,9 +1613,16 @@ zpool_in_use(libzfs_handle_t *hdl, int f
                 * its state to active.
                 */
                if (pool_active(hdl, name, guid, &isactive) == 0 && isactive &&
-                   (zhp = zpool_open_canfail(hdl, name)) != NULL &&
-                   zpool_get_prop_int(zhp, ZPOOL_PROP_READONLY, NULL))
-                       stateval = POOL_STATE_ACTIVE;
+                   (zhp = zpool_open_canfail(hdl, name)) != NULL) {
+                       if (zpool_get_prop_int(zhp, ZPOOL_PROP_READONLY, NULL))
+                               stateval = POOL_STATE_ACTIVE;
+
+                       /*
+                        * All we needed the zpool handle for is the
+                        * readonly prop check.
+                        */
+                       zpool_close(zhp);
+               }
 
                ret = B_TRUE;
                break;
_______________________________________________
svn-src-stable-9@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-stable-9
To unsubscribe, send any mail to "svn-src-stable-9-unsubscr...@freebsd.org"

Reply via email to