aboutsummaryrefslogtreecommitdiffstats
path: root/lib/libzfs/os
diff options
context:
space:
mode:
authorDon Brady <[email protected]>2021-03-08 09:46:45 -0700
committerGitHub <[email protected]>2021-03-08 08:46:45 -0800
commitf5ada6538dbc07f4589cbeff8f84228d1575bb03 (patch)
tree0f1d2660720737d874ac18166a5f45c100936560 /lib/libzfs/os
parent4fdbd434508fac4013b856d804f9f191baea5690 (diff)
Return finer grain errors in libzfs unmount_one
Added errno mappings to unmount_one() in libzfs. Changed do_unmount() implementation to return errno errors directly like is done for do_mount() and others. Reviewed-by: Mark Maybee <[email protected]> Reviewed-by: Brian Behlendorf <[email protected]> Signed-off-by: Don Brady <[email protected]> Closes #11681
Diffstat (limited to 'lib/libzfs/os')
-rw-r--r--lib/libzfs/os/freebsd/libzfs_zmount.c5
-rw-r--r--lib/libzfs/os/linux/libzfs_mount_os.c6
2 files changed, 7 insertions, 4 deletions
diff --git a/lib/libzfs/os/freebsd/libzfs_zmount.c b/lib/libzfs/os/freebsd/libzfs_zmount.c
index 2207fffc5..e114b1e0c 100644
--- a/lib/libzfs/os/freebsd/libzfs_zmount.c
+++ b/lib/libzfs/os/freebsd/libzfs_zmount.c
@@ -128,8 +128,9 @@ do_mount(zfs_handle_t *zhp, const char *mntpt, char *opts, int flags)
int
do_unmount(const char *mntpt, int flags)
{
-
- return (unmount(mntpt, flags));
+ if (unmount(mntpt, flags) < 0)
+ return (errno);
+ return (0);
}
int
diff --git a/lib/libzfs/os/linux/libzfs_mount_os.c b/lib/libzfs/os/linux/libzfs_mount_os.c
index 92052f28f..21d640538 100644
--- a/lib/libzfs/os/linux/libzfs_mount_os.c
+++ b/lib/libzfs/os/linux/libzfs_mount_os.c
@@ -22,7 +22,7 @@
/*
* Copyright 2015 Nexenta Systems, Inc. All rights reserved.
* Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
- * Copyright (c) 2014, 2020 by Delphix. All rights reserved.
+ * Copyright (c) 2014, 2021 by Delphix. All rights reserved.
* Copyright 2016 Igor Kozhukhov <[email protected]>
* Copyright 2017 RackTop Systems.
* Copyright (c) 2018 Datto Inc.
@@ -377,7 +377,9 @@ int
do_unmount(const char *mntpt, int flags)
{
if (!libzfs_envvar_is_set("ZFS_MOUNT_HELPER")) {
- return (umount2(mntpt, flags));
+ int rv = umount2(mntpt, flags);
+
+ return (rv < 0 ? errno : 0);
}
char force_opt[] = "-f";