Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 25 Jul 2018 15:45:42 +0000 (UTC)
From:      Kyle Evans <kevans@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject:   svn commit: r336713 - projects/bectl/lib/libbe
Message-ID:  <201807251545.w6PFjgGK052973@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kevans
Date: Wed Jul 25 15:45:42 2018
New Revision: 336713
URL: https://svnweb.freebsd.org/changeset/base/336713

Log:
  libbe(3): Use zfs_is_mounted to check mounted and mountpoint

Modified:
  projects/bectl/lib/libbe/be_info.c

Modified: projects/bectl/lib/libbe/be_info.c
==============================================================================
--- projects/bectl/lib/libbe/be_info.c	Wed Jul 25 15:44:49 2018	(r336712)
+++ projects/bectl/lib/libbe/be_info.c	Wed Jul 25 15:45:42 2018	(r336713)
@@ -114,7 +114,7 @@ be_get_bootenv_props(libbe_handle_t *lbh, nvlist_t *ds
 static int
 prop_list_builder_cb(zfs_handle_t *zfs_hdl, void *data_p)
 {
-	char buf[512];
+	char buf[512], *mountpoint;
 	prop_data_t *data;
 	libbe_handle_t *lbh;
 	nvlist_t *props;
@@ -137,15 +137,11 @@ prop_list_builder_cb(zfs_handle_t *zfs_hdl, void *data
 	name = strrchr(dataset, '/') + 1;
 	nvlist_add_string(props, "name", name);
 
-	mounted = zfs_prop_get_int(zfs_hdl, ZFS_PROP_MOUNTED);
+	mounted = zfs_is_mounted(zfs_hdl, &mountpoint);
 	nvlist_add_boolean_value(props, "mounted", mounted);
 
-	/* XXX TODO: NOT CORRECT! Must use is_mounted */
-	if (mounted) {
-		zfs_prop_get(zfs_hdl, ZFS_PROP_MOUNTPOINT, buf, 512,
-		    NULL, NULL, 0, 1);
-		nvlist_add_string(props, "mountpoint", buf);
-	}
+	if (mounted)
+		nvlist_add_string(props, "mountpoint", mountpoint);
 
 	if (zfs_prop_get(zfs_hdl, ZFS_PROP_ORIGIN, buf, 512,
 	    NULL, NULL, 0, 1))



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201807251545.w6PFjgGK052973>