From 659f4008be86d5eaf67184c83fddbcb6800a3705 Mon Sep 17 00:00:00 2001 From: Ryan Moeller Date: Thu, 9 Jul 2020 20:47:12 -0400 Subject: [PATCH] libzfs: Make zfs_cmd_t initialization consistent, use zfs_ioctl The clang version 8.0.1 shipped in FreeBSD 12.1-RELEASE also oddly throws a warning that is treated as an error on the initialization of the zc struct in zpool_nextboot. The zpool_nextboot code from FreeBSD was not updated to use zfs_ioctl. Switch ioctl to zfs_ioctl in and use {"\0"} to initialize the struct. Do a consistency pass for zfs_cmd_t initialization. Reviewed-by: Brian Behlendorf Reviewed-by: George Wilson Signed-off-by: Ryan Moeller Closes #10539 --- lib/libzfs/libzfs_sendrecv.c | 3 ++- lib/libzfs/os/freebsd/libzfs_compat.c | 6 +++--- lib/libzfs_core/libzfs_core.c | 5 +++-- 3 files changed, 8 insertions(+), 6 deletions(-) diff --git a/lib/libzfs/libzfs_sendrecv.c b/lib/libzfs/libzfs_sendrecv.c index 75f4e056b6..f1524c384f 100644 --- a/lib/libzfs/libzfs_sendrecv.c +++ b/lib/libzfs/libzfs_sendrecv.c @@ -860,7 +860,8 @@ int zfs_send_progress(zfs_handle_t *zhp, int fd, uint64_t *bytes_written, uint64_t *blocks_visited) { - zfs_cmd_t zc = { {0} }; + zfs_cmd_t zc = {"\0"}; + (void) strlcpy(zc.zc_name, zhp->zfs_name, sizeof (zc.zc_name)); zc.zc_cookie = fd; if (zfs_ioctl(zhp->zfs_hdl, ZFS_IOC_SEND_PROGRESS, &zc) != 0) diff --git a/lib/libzfs/os/freebsd/libzfs_compat.c b/lib/libzfs/os/freebsd/libzfs_compat.c index 50a3c7b77c..98269b57b7 100644 --- a/lib/libzfs/os/freebsd/libzfs_compat.c +++ b/lib/libzfs/os/freebsd/libzfs_compat.c @@ -230,7 +230,7 @@ int zfs_jail(zfs_handle_t *zhp, int jailid, int attach) { libzfs_handle_t *hdl = zhp->zfs_hdl; - zfs_cmd_t zc = { { 0 } }; + zfs_cmd_t zc = {"\0"}; char errbuf[1024]; unsigned long cmd; int ret; @@ -281,7 +281,7 @@ int zpool_nextboot(libzfs_handle_t *hdl, uint64_t pool_guid, uint64_t dev_guid, const char *command) { - zfs_cmd_t zc = { 0 }; + zfs_cmd_t zc = {"\0"}; nvlist_t *args; int error; @@ -291,7 +291,7 @@ zpool_nextboot(libzfs_handle_t *hdl, uint64_t pool_guid, uint64_t dev_guid, fnvlist_add_string(args, "command", command); error = zcmd_write_src_nvlist(hdl, &zc, args); if (error == 0) - error = ioctl(hdl->libzfs_fd, ZFS_IOC_NEXTBOOT, &zc); + error = zfs_ioctl(hdl, ZFS_IOC_NEXTBOOT, &zc); zcmd_free_nvlists(&zc); nvlist_free(args); return (error); diff --git a/lib/libzfs_core/libzfs_core.c b/lib/libzfs_core/libzfs_core.c index 22996fc9be..d35ea523e0 100644 --- a/lib/libzfs_core/libzfs_core.c +++ b/lib/libzfs_core/libzfs_core.c @@ -293,7 +293,7 @@ lzc_promote(const char *fsname, char *snapnamebuf, int snapnamelen) * The promote ioctl is still legacy, so we need to construct our * own zfs_cmd_t rather than using lzc_ioctl(). */ - zfs_cmd_t zc = { "\0" }; + zfs_cmd_t zc = {"\0"}; ASSERT3S(g_refcount, >, 0); VERIFY3S(g_fd, !=, -1); @@ -311,8 +311,9 @@ lzc_promote(const char *fsname, char *snapnamebuf, int snapnamelen) int lzc_rename(const char *source, const char *target) { - zfs_cmd_t zc = { "\0" }; + zfs_cmd_t zc = {"\0"}; int error; + ASSERT3S(g_refcount, >, 0); VERIFY3S(g_fd, !=, -1); (void) strlcpy(zc.zc_name, source, sizeof (zc.zc_name));