lib/libzfs/libzfs_changelist.c

   1 /*
   2  * CDDL HEADER START
   3  *
   4  * The contents of this file are subject to the terms of the
   5  * Common Development and Distribution License (the "License").
   6  * You may not use this file except in compliance with the License.
   7  *
   8  * You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
   9  * or http://www.opensolaris.org/os/licensing.
  10  * See the License for the specific language governing permissions
  11  * and limitations under the License.
  12  *
  13  * When distributing Covered Code, include this CDDL HEADER in each
  14  * file and include the License file at usr/src/OPENSOLARIS.LICENSE.
  15  * If applicable, add the following below this CDDL HEADER, with the
  16  * fields enclosed by brackets "[]" replaced with your own identifying
  17  * information: Portions Copyright [yyyy] [name of copyright owner]
  18  *
  19  * CDDL HEADER END
  20  */
  21
  22 /*
  23  * Copyright 2010 Sun Microsystems, Inc.  All rights reserved.
  24  * Use is subject to license terms.
  25  *
  26  * Portions Copyright 2007 Ramprakash Jelari
  27  */
  28
  29 #include <libintl.h>
  30 #include <libuutil.h>
  31 #include <stddef.h>
  32 #include <stdlib.h>
  33 #include <string.h>
  34 #include <unistd.h>
  35 #include <zone.h>
  36
  37 #include <libzfs.h>
  38
  39 #include "libzfs_impl.h"
  40
  41 /*
  42  * Structure to keep track of dataset state.  Before changing the 'sharenfs' or
  43  * 'mountpoint' property, we record whether the filesystem was previously
  44  * mounted/shared.  This prior state dictates whether we remount/reshare the
  45  * dataset after the property has been changed.
  46  *
  47  * The interface consists of the following sequence of functions:
  48  *
  49  *      changelist_gather()
  50  *      changelist_prefix()
  51  *      < change property >
  52  *      changelist_postfix()
  53  *      changelist_free()
  54  *
  55  * Other interfaces:
  56  *
  57  * changelist_remove() - remove a node from a gathered list
  58  * changelist_rename() - renames all datasets appropriately when doing a rename
  59  * changelist_unshare() - unshares all the nodes in a given changelist
  60  * changelist_haszonedchild() - check if there is any child exported to
  61  *                              a local zone
  62  */
  63 typedef struct prop_changenode {
  64         zfs_handle_t            *cn_handle;
  65         int                     cn_shared;
  66         int                     cn_mounted;
  67         int                     cn_zoned;
  68         boolean_t               cn_needpost;    /* is postfix() needed? */
  69         uu_list_node_t          cn_listnode;
  70 } prop_changenode_t;
  71
  72 struct prop_changelist {
  73         zfs_prop_t              cl_prop;
  74         zfs_prop_t              cl_realprop;
  75         zfs_prop_t              cl_shareprop;  /* used with sharenfs/sharesmb */
  76         uu_list_pool_t          *cl_pool;
  77         uu_list_t               *cl_list;
  78         boolean_t               cl_waslegacy;
  79         boolean_t               cl_allchildren;
  80         boolean_t               cl_alldependents;
  81         int                     cl_mflags;      /* Mount flags */
  82         int                     cl_gflags;      /* Gather request flags */
  83         boolean_t               cl_haszonedchild;
  84         boolean_t               cl_sorted;
  85 };
  86
  87 /*
  88  * If the property is 'mountpoint', go through and unmount filesystems as
  89  * necessary.  We don't do the same for 'sharenfs', because we can just re-share
  90  * with different options without interrupting service. We do handle 'sharesmb'
  91  * since there may be old resource names that need to be removed.
  92  */
  93 int
  94 changelist_prefix(prop_changelist_t *clp)
  95 {
  96         prop_changenode_t *cn;
  97         int ret = 0;
  98
  99         if (clp->cl_prop != ZFS_PROP_MOUNTPOINT &&
 100             clp->cl_prop != ZFS_PROP_SHARESMB)
 101                 return (0);
 102
 103         for (cn = uu_list_first(clp->cl_list); cn != NULL;
 104             cn = uu_list_next(clp->cl_list, cn)) {
 105
 106                 /* if a previous loop failed, set the remaining to false */
 107                 if (ret == -1) {
 108                         cn->cn_needpost = B_FALSE;
 109                         continue;
 110                 }
 111
 112                 /*
 113                  * If we are in the global zone, but this dataset is exported
 114                  * to a local zone, do nothing.
 115                  */
 116                 if (getzoneid() == GLOBAL_ZONEID && cn->cn_zoned)
 117                         continue;
 118
 119                 if (!ZFS_IS_VOLUME(cn->cn_handle)) {
 120                         /*
 121                          * Do the property specific processing.
 122                          */
 123                         switch (clp->cl_prop) {
 124                         case ZFS_PROP_MOUNTPOINT:
 125                                 if (zfs_unmount(cn->cn_handle, NULL,
 126                                     clp->cl_mflags) != 0) {
 127                                         ret = -1;
 128                                         cn->cn_needpost = B_FALSE;
 129                                 }
 130                                 break;
 131                         case ZFS_PROP_SHARESMB:
 132                                 (void) zfs_unshare_smb(cn->cn_handle, NULL);
 133                                 break;
 134                         default:
 135                                 break;
 136                         }
 137                 }
 138         }
 139
 140         if (ret == -1)
 141                 (void) changelist_postfix(clp);
 142
 143         return (ret);
 144 }
 145
 146 /*
 147  * If the property is 'mountpoint' or 'sharenfs', go through and remount and/or
 148  * reshare the filesystems as necessary.  In changelist_gather() we recorded
 149  * whether the filesystem was previously shared or mounted.  The action we take
 150  * depends on the previous state, and whether the value was previously 'legacy'.
 151  * For non-legacy properties, we only remount/reshare the filesystem if it was
 152  * previously mounted/shared.  Otherwise, we always remount/reshare the
 153  * filesystem.
 154  */
 155 int
 156 changelist_postfix(prop_changelist_t *clp)
 157 {
 158         prop_changenode_t *cn;
 159         char shareopts[ZFS_MAXPROPLEN];
 160         int errors = 0;
 161         libzfs_handle_t *hdl;
 162
 163         /*
 164          * If we're changing the mountpoint, attempt to destroy the underlying
 165          * mountpoint.  All other datasets will have inherited from this dataset
 166          * (in which case their mountpoints exist in the filesystem in the new
 167          * location), or have explicit mountpoints set (in which case they won't
 168          * be in the changelist).
 169          */
 170         if ((cn = uu_list_last(clp->cl_list)) == NULL)
 171                 return (0);
 172
 173         if (clp->cl_prop == ZFS_PROP_MOUNTPOINT)
 174                 remove_mountpoint(cn->cn_handle);
 175
 176         /*
 177          * It is possible that the changelist_prefix() used libshare
 178          * to unshare some entries. Since libshare caches data, an
 179          * attempt to reshare during postfix can fail unless libshare
 180          * is uninitialized here so that it will reinitialize later.
 181          */
 182         if (cn->cn_handle != NULL) {
 183                 hdl = cn->cn_handle->zfs_hdl;
 184                 assert(hdl != NULL);
 185                 zfs_uninit_libshare(hdl);
 186         }
 187
 188         /*
 189          * We walk the datasets in reverse, because we want to mount any parent
 190          * datasets before mounting the children.  We walk all datasets even if
 191          * there are errors.
 192          */
 193         for (cn = uu_list_last(clp->cl_list); cn != NULL;
 194             cn = uu_list_prev(clp->cl_list, cn)) {
 195
 196                 boolean_t sharenfs;
 197                 boolean_t sharesmb;
 198                 boolean_t mounted;
 199
 200                 /*
 201                  * If we are in the global zone, but this dataset is exported
 202                  * to a local zone, do nothing.
 203                  */
 204                 if (getzoneid() == GLOBAL_ZONEID && cn->cn_zoned)
 205                         continue;
 206
 207                 /* Only do post-processing if it's required */
 208                 if (!cn->cn_needpost)
 209                         continue;
 210                 cn->cn_needpost = B_FALSE;
 211
 212                 zfs_refresh_properties(cn->cn_handle);
 213
 214                 if (ZFS_IS_VOLUME(cn->cn_handle))
 215                         continue;
 216
 217                 /*
 218                  * Remount if previously mounted or mountpoint was legacy,
 219                  * or sharenfs or sharesmb  property is set.
 220                  */
 221                 sharenfs = ((zfs_prop_get(cn->cn_handle, ZFS_PROP_SHARENFS,
 222                     shareopts, sizeof (shareopts), NULL, NULL, 0,
 223                     B_FALSE) == 0) && (strcmp(shareopts, "off") != 0));
 224
 225                 sharesmb = ((zfs_prop_get(cn->cn_handle, ZFS_PROP_SHARESMB,
 226                     shareopts, sizeof (shareopts), NULL, NULL, 0,
 227                     B_FALSE) == 0) && (strcmp(shareopts, "off") != 0));
 228
 229                 mounted = zfs_is_mounted(cn->cn_handle, NULL);
 230
 231                 if (!mounted && (cn->cn_mounted ||
 232                     ((sharenfs || sharesmb || clp->cl_waslegacy) &&
 233                     (zfs_prop_get_int(cn->cn_handle,
 234                     ZFS_PROP_CANMOUNT) == ZFS_CANMOUNT_ON)))) {
 235
 236                         if (zfs_mount(cn->cn_handle, NULL, 0) != 0)
 237                                 errors++;
 238                         else
 239                                 mounted = TRUE;
 240                 }
 241
 242                 /*
 243                  * If the file system is mounted we always re-share even
 244                  * if the filesystem is currently shared, so that we can
 245                  * adopt any new options.
 246                  */
 247                 if (sharenfs && mounted)
 248                         errors += zfs_share_nfs(cn->cn_handle);
 249                 else if (cn->cn_shared || clp->cl_waslegacy)
 250                         errors += zfs_unshare_nfs(cn->cn_handle, NULL);
 251                 if (sharesmb && mounted)
 252                         errors += zfs_share_smb(cn->cn_handle);
 253                 else if (cn->cn_shared || clp->cl_waslegacy)
 254                         errors += zfs_unshare_smb(cn->cn_handle, NULL);
 255         }
 256
 257         return (errors ? -1 : 0);
 258 }
 259
 260 /*
 261  * Is this "dataset" a child of "parent"?
 262  */
 263 boolean_t
 264 isa_child_of(const char *dataset, const char *parent)
 265 {
 266         int len;
 267
 268         len = strlen(parent);
 269
 270         if (strncmp(dataset, parent, len) == 0 &&
 271             (dataset[len] == '@' || dataset[len] == '/' ||
 272             dataset[len] == '\0'))
 273                 return (B_TRUE);
 274         else
 275                 return (B_FALSE);
 276
 277 }
 278
 279 /*
 280  * If we rename a filesystem, child filesystem handles are no longer valid
 281  * since we identify each dataset by its name in the ZFS namespace.  As a
 282  * result, we have to go through and fix up all the names appropriately.  We
 283  * could do this automatically if libzfs kept track of all open handles, but
 284  * this is a lot less work.
 285  */
 286 void
 287 changelist_rename(prop_changelist_t *clp, const char *src, const char *dst)
 288 {
 289         prop_changenode_t *cn;
 290         char newname[ZFS_MAXNAMELEN];
 291
 292         for (cn = uu_list_first(clp->cl_list); cn != NULL;
 293             cn = uu_list_next(clp->cl_list, cn)) {
 294                 zfs_handle_t *hdl;
 295
 296                 hdl = cn->cn_handle;
 297
 298                 /*
 299                  * Do not rename a clone that's not in the source hierarchy.
 300                  */
 301                 if (!isa_child_of(hdl->zfs_name, src))
 302                         continue;
 303
 304                 /*
 305                  * Destroy the previous mountpoint if needed.
 306                  */
 307                 remove_mountpoint(hdl);
 308
 309                 (void) strlcpy(newname, dst, sizeof (newname));
 310                 (void) strcat(newname, hdl->zfs_name + strlen(src));
 311
 312                 if (ZFS_IS_VOLUME(hdl)) {
 313                         (void) zvol_remove_link(hdl->zfs_hdl, hdl->zfs_name);
 314                         (void) zvol_create_link(hdl->zfs_hdl, newname);
 315                 }
 316
 317                 (void) strlcpy(hdl->zfs_name, newname, sizeof (hdl->zfs_name));
 318         }
 319 }
 320
 321 /*
 322  * Given a gathered changelist for the 'sharenfs' or 'sharesmb' property,
 323  * unshare all the datasets in the list.
 324  */
 325 int
 326 changelist_unshare(prop_changelist_t *clp, zfs_share_proto_t *proto)
 327 {
 328         prop_changenode_t *cn;
 329         int ret = 0;
 330
 331         if (clp->cl_prop != ZFS_PROP_SHARENFS &&
 332             clp->cl_prop != ZFS_PROP_SHARESMB)
 333                 return (0);
 334
 335         for (cn = uu_list_first(clp->cl_list); cn != NULL;
 336             cn = uu_list_next(clp->cl_list, cn)) {
 337                 if (zfs_unshare_proto(cn->cn_handle, NULL, proto) != 0)
 338                         ret = -1;
 339         }
 340
 341         return (ret);
 342 }
 343
 344 /*
 345  * Check if there is any child exported to a local zone in a given changelist.
 346  * This information has already been recorded while gathering the changelist
 347  * via changelist_gather().
 348  */
 349 int
 350 changelist_haszonedchild(prop_changelist_t *clp)
 351 {
 352         return (clp->cl_haszonedchild);
 353 }
 354
 355 /*
 356  * Remove a node from a gathered list.
 357  */
 358 void
 359 changelist_remove(prop_changelist_t *clp, const char *name)
 360 {
 361         prop_changenode_t *cn;
 362
 363         for (cn = uu_list_first(clp->cl_list); cn != NULL;
 364             cn = uu_list_next(clp->cl_list, cn)) {
 365
 366                 if (strcmp(cn->cn_handle->zfs_name, name) == 0) {
 367                         uu_list_remove(clp->cl_list, cn);
 368                         zfs_close(cn->cn_handle);
 369                         free(cn);
 370                         return;
 371                 }
 372         }
 373 }
 374
 375 /*
 376  * Release any memory associated with a changelist.
 377  */
 378 void
 379 changelist_free(prop_changelist_t *clp)
 380 {
 381         prop_changenode_t *cn;
 382         void *cookie;
 383
 384         if (clp->cl_list) {
 385                 cookie = NULL;
 386                 while ((cn = uu_list_teardown(clp->cl_list, &cookie)) != NULL) {
 387                         zfs_close(cn->cn_handle);
 388                         free(cn);
 389                 }
 390
 391                 uu_list_destroy(clp->cl_list);
 392         }
 393         if (clp->cl_pool)
 394                 uu_list_pool_destroy(clp->cl_pool);
 395
 396         free(clp);
 397 }
 398
 399 static int
 400 change_one(zfs_handle_t *zhp, void *data)
 401 {
 402         prop_changelist_t *clp = data;
 403         char property[ZFS_MAXPROPLEN];
 404         char where[64];
 405         prop_changenode_t *cn;
 406         zprop_source_t sourcetype;
 407         zprop_source_t share_sourcetype;
 408
 409         /*
 410          * We only want to unmount/unshare those filesystems that may inherit
 411          * from the target filesystem.  If we find any filesystem with a
 412          * locally set mountpoint, we ignore any children since changing the
 413          * property will not affect them.  If this is a rename, we iterate
 414          * over all children regardless, since we need them unmounted in
 415          * order to do the rename.  Also, if this is a volume and we're doing
 416          * a rename, then always add it to the changelist.
 417          */
 418
 419         if (!(ZFS_IS_VOLUME(zhp) && clp->cl_realprop == ZFS_PROP_NAME) &&
 420             zfs_prop_get(zhp, clp->cl_prop, property,
 421             sizeof (property), &sourcetype, where, sizeof (where),
 422             B_FALSE) != 0) {
 423                 zfs_close(zhp);
 424                 return (0);
 425         }
 426
 427         /*
 428          * If we are "watching" sharenfs or sharesmb
 429          * then check out the companion property which is tracked
 430          * in cl_shareprop
 431          */
 432         if (clp->cl_shareprop != ZPROP_INVAL &&
 433             zfs_prop_get(zhp, clp->cl_shareprop, property,
 434             sizeof (property), &share_sourcetype, where, sizeof (where),
 435             B_FALSE) != 0) {
 436                 zfs_close(zhp);
 437                 return (0);
 438         }
 439
 440         if (clp->cl_alldependents || clp->cl_allchildren ||
 441             sourcetype == ZPROP_SRC_DEFAULT ||
 442             sourcetype == ZPROP_SRC_INHERITED ||
 443             (clp->cl_shareprop != ZPROP_INVAL &&
 444             (share_sourcetype == ZPROP_SRC_DEFAULT ||
 445             share_sourcetype == ZPROP_SRC_INHERITED))) {
 446                 if ((cn = zfs_alloc(zfs_get_handle(zhp),
 447                     sizeof (prop_changenode_t))) == NULL) {
 448                         zfs_close(zhp);
 449                         return (-1);
 450                 }
 451
 452                 cn->cn_handle = zhp;
 453                 cn->cn_mounted = (clp->cl_gflags & CL_GATHER_MOUNT_ALWAYS) ||
 454                     zfs_is_mounted(zhp, NULL);
 455                 cn->cn_shared = zfs_is_shared(zhp);
 456                 cn->cn_zoned = zfs_prop_get_int(zhp, ZFS_PROP_ZONED);
 457                 cn->cn_needpost = B_TRUE;
 458
 459                 /* Indicate if any child is exported to a local zone. */
 460                 if (getzoneid() == GLOBAL_ZONEID && cn->cn_zoned)
 461                         clp->cl_haszonedchild = B_TRUE;
 462
 463                 uu_list_node_init(cn, &cn->cn_listnode, clp->cl_pool);
 464
 465                 if (clp->cl_sorted) {
 466                         uu_list_index_t idx;
 467
 468                         (void) uu_list_find(clp->cl_list, cn, NULL,
 469                             &idx);
 470                         uu_list_insert(clp->cl_list, cn, idx);
 471                 } else {
 472                         /*
 473                          * Add this child to beginning of the list. Children
 474                          * below this one in the hierarchy will get added above
 475                          * this one in the list. This produces a list in
 476                          * reverse dataset name order.
 477                          * This is necessary when the original mountpoint
 478                          * is legacy or none.
 479                          */
 480                         ASSERT(!clp->cl_alldependents);
 481                         verify(uu_list_insert_before(clp->cl_list,
 482                             uu_list_first(clp->cl_list), cn) == 0);
 483                 }
 484
 485                 if (!clp->cl_alldependents)
 486                         return (zfs_iter_children(zhp, change_one, data));
 487         } else {
 488                 zfs_close(zhp);
 489         }
 490
 491         return (0);
 492 }
 493
 494 /*ARGSUSED*/
 495 static int
 496 compare_mountpoints(const void *a, const void *b, void *unused)
 497 {
 498         const prop_changenode_t *ca = a;
 499         const prop_changenode_t *cb = b;
 500
 501         char mounta[MAXPATHLEN];
 502         char mountb[MAXPATHLEN];
 503
 504         boolean_t hasmounta, hasmountb;
 505
 506         /*
 507          * When unsharing or unmounting filesystems, we need to do it in
 508          * mountpoint order.  This allows the user to have a mountpoint
 509          * hierarchy that is different from the dataset hierarchy, and still
 510          * allow it to be changed.  However, if either dataset doesn't have a
 511          * mountpoint (because it is a volume or a snapshot), we place it at the
 512          * end of the list, because it doesn't affect our change at all.
 513          */
 514         hasmounta = (zfs_prop_get(ca->cn_handle, ZFS_PROP_MOUNTPOINT, mounta,
 515             sizeof (mounta), NULL, NULL, 0, B_FALSE) == 0);
 516         hasmountb = (zfs_prop_get(cb->cn_handle, ZFS_PROP_MOUNTPOINT, mountb,
 517             sizeof (mountb), NULL, NULL, 0, B_FALSE) == 0);
 518
 519         if (!hasmounta && hasmountb)
 520                 return (-1);
 521         else if (hasmounta && !hasmountb)
 522                 return (1);
 523         else if (!hasmounta && !hasmountb)
 524                 return (0);
 525         else
 526                 return (strcmp(mountb, mounta));
 527 }
 528
 529 /*
 530  * Given a ZFS handle and a property, construct a complete list of datasets
 531  * that need to be modified as part of this process.  For anything but the
 532  * 'mountpoint' and 'sharenfs' properties, this just returns an empty list.
 533  * Otherwise, we iterate over all children and look for any datasets that
 534  * inherit the property.  For each such dataset, we add it to the list and
 535  * mark whether it was shared beforehand.
 536  */
 537 prop_changelist_t *
 538 changelist_gather(zfs_handle_t *zhp, zfs_prop_t prop, int gather_flags,
 539     int mnt_flags)
 540 {
 541         prop_changelist_t *clp;
 542         prop_changenode_t *cn;
 543         zfs_handle_t *temp;
 544         char property[ZFS_MAXPROPLEN];
 545         uu_compare_fn_t *compare = NULL;
 546         boolean_t legacy = B_FALSE;
 547
 548         if ((clp = zfs_alloc(zhp->zfs_hdl, sizeof (prop_changelist_t))) == NULL)
 549                 return (NULL);
 550
 551         /*
 552          * For mountpoint-related tasks, we want to sort everything by
 553          * mountpoint, so that we mount and unmount them in the appropriate
 554          * order, regardless of their position in the hierarchy.
 555          */
 556         if (prop == ZFS_PROP_NAME || prop == ZFS_PROP_ZONED ||
 557             prop == ZFS_PROP_MOUNTPOINT || prop == ZFS_PROP_SHARENFS ||
 558             prop == ZFS_PROP_SHARESMB) {
 559
 560                 if (zfs_prop_get(zhp, ZFS_PROP_MOUNTPOINT,
 561                     property, sizeof (property),
 562                     NULL, NULL, 0, B_FALSE) == 0 &&
 563                     (strcmp(property, "legacy") == 0 ||
 564                     strcmp(property, "none") == 0)) {
 565
 566                         legacy = B_TRUE;
 567                 }
 568                 if (!legacy) {
 569                         compare = compare_mountpoints;
 570                         clp->cl_sorted = B_TRUE;
 571                 }
 572         }
 573
 574         clp->cl_pool = uu_list_pool_create("changelist_pool",
 575             sizeof (prop_changenode_t),
 576             offsetof(prop_changenode_t, cn_listnode),
 577             compare, 0);
 578         if (clp->cl_pool == NULL) {
 579                 assert(uu_error() == UU_ERROR_NO_MEMORY);
 580                 (void) zfs_error(zhp->zfs_hdl, EZFS_NOMEM, "internal error");
 581                 changelist_free(clp);
 582                 return (NULL);
 583         }
 584
 585         clp->cl_list = uu_list_create(clp->cl_pool, NULL,
 586             clp->cl_sorted ? UU_LIST_SORTED : 0);
 587         clp->cl_gflags = gather_flags;
 588         clp->cl_mflags = mnt_flags;
 589
 590         if (clp->cl_list == NULL) {
 591                 assert(uu_error() == UU_ERROR_NO_MEMORY);
 592                 (void) zfs_error(zhp->zfs_hdl, EZFS_NOMEM, "internal error");
 593                 changelist_free(clp);
 594                 return (NULL);
 595         }
 596
 597         /*
 598          * If this is a rename or the 'zoned' property, we pretend we're
 599          * changing the mountpoint and flag it so we can catch all children in
 600          * change_one().
 601          *
 602          * Flag cl_alldependents to catch all children plus the dependents
 603          * (clones) that are not in the hierarchy.
 604          */
 605         if (prop == ZFS_PROP_NAME) {
 606                 clp->cl_prop = ZFS_PROP_MOUNTPOINT;
 607                 clp->cl_alldependents = B_TRUE;
 608         } else if (prop == ZFS_PROP_ZONED) {
 609                 clp->cl_prop = ZFS_PROP_MOUNTPOINT;
 610                 clp->cl_allchildren = B_TRUE;
 611         } else if (prop == ZFS_PROP_CANMOUNT) {
 612                 clp->cl_prop = ZFS_PROP_MOUNTPOINT;
 613         } else if (prop == ZFS_PROP_VOLSIZE) {
 614                 clp->cl_prop = ZFS_PROP_MOUNTPOINT;
 615         } else {
 616                 clp->cl_prop = prop;
 617         }
 618         clp->cl_realprop = prop;
 619
 620         if (clp->cl_prop != ZFS_PROP_MOUNTPOINT &&
 621             clp->cl_prop != ZFS_PROP_SHARENFS &&
 622             clp->cl_prop != ZFS_PROP_SHARESMB)
 623                 return (clp);
 624
 625         /*
 626          * If watching SHARENFS or SHARESMB then
 627          * also watch its companion property.
 628          */
 629         if (clp->cl_prop == ZFS_PROP_SHARENFS)
 630                 clp->cl_shareprop = ZFS_PROP_SHARESMB;
 631         else if (clp->cl_prop == ZFS_PROP_SHARESMB)
 632                 clp->cl_shareprop = ZFS_PROP_SHARENFS;
 633
 634         if (clp->cl_alldependents) {
 635                 if (zfs_iter_dependents(zhp, B_TRUE, change_one, clp) != 0) {
 636                         changelist_free(clp);
 637                         return (NULL);
 638                 }
 639         } else if (zfs_iter_children(zhp, change_one, clp) != 0) {
 640                 changelist_free(clp);
 641                 return (NULL);
 642         }
 643
 644         /*
 645          * We have to re-open ourselves because we auto-close all the handles
 646          * and can't tell the difference.
 647          */
 648         if ((temp = zfs_open(zhp->zfs_hdl, zfs_get_name(zhp),
 649             ZFS_TYPE_DATASET)) == NULL) {
 650                 changelist_free(clp);
 651                 return (NULL);
 652         }
 653
 654         /*
 655          * Always add ourself to the list.  We add ourselves to the end so that
 656          * we're the last to be unmounted.
 657          */
 658         if ((cn = zfs_alloc(zhp->zfs_hdl,
 659             sizeof (prop_changenode_t))) == NULL) {
 660                 zfs_close(temp);
 661                 changelist_free(clp);
 662                 return (NULL);
 663         }
 664
 665         cn->cn_handle = temp;
 666         cn->cn_mounted = (clp->cl_gflags & CL_GATHER_MOUNT_ALWAYS) ||
 667             zfs_is_mounted(temp, NULL);
 668         cn->cn_shared = zfs_is_shared(temp);
 669         cn->cn_zoned = zfs_prop_get_int(zhp, ZFS_PROP_ZONED);
 670         cn->cn_needpost = B_TRUE;
 671
 672         uu_list_node_init(cn, &cn->cn_listnode, clp->cl_pool);
 673         if (clp->cl_sorted) {
 674                 uu_list_index_t idx;
 675                 (void) uu_list_find(clp->cl_list, cn, NULL, &idx);
 676                 uu_list_insert(clp->cl_list, cn, idx);
 677         } else {
 678                 /*
 679                  * Add the target dataset to the end of the list.
 680                  * The list is not really unsorted. The list will be
 681                  * in reverse dataset name order. This is necessary
 682                  * when the original mountpoint is legacy or none.
 683                  */
 684                 verify(uu_list_insert_after(clp->cl_list,
 685                     uu_list_last(clp->cl_list), cn) == 0);
 686         }
 687
 688         /*
 689          * If the mountpoint property was previously 'legacy', or 'none',
 690          * record it as the behavior of changelist_postfix() will be different.
 691          */
 692         if ((clp->cl_prop == ZFS_PROP_MOUNTPOINT) && legacy) {
 693                 /*
 694                  * do not automatically mount ex-legacy datasets if
 695                  * we specifically set canmount to noauto
 696                  */
 697                 if (zfs_prop_get_int(zhp, ZFS_PROP_CANMOUNT) !=
 698                     ZFS_CANMOUNT_NOAUTO)
 699                         clp->cl_waslegacy = B_TRUE;
 700         }
 701
 702         return (clp);
 703 }