Stop segfaulting on unmount error case

After interrupting ZTS runs that errored out, I found that
"zpool export testpool2" was segfaulting.

This seems unnecessary.

Reviewed-by: szubersk <szuberskidamian@gmail.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Jorgen Lundman <lundman@lundman.net>
Signed-off-by: Rich Ercolani <rincebrain@gmail.com>
Closes #12804
This commit is contained in:
Rich Ercolani 2021-11-30 13:36:36 -05:00 committed by GitHub
parent 547df81641
commit 5dc6fc2b73
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 3 deletions

View File

@ -595,9 +595,13 @@ unmount_one(zfs_handle_t *zhp, const char *mountpoint, int flags)
default: default:
libzfs_err = EZFS_UMOUNTFAILED; libzfs_err = EZFS_UMOUNTFAILED;
} }
return (zfs_error_fmt(zhp->zfs_hdl, libzfs_err, if (zhp) {
dgettext(TEXT_DOMAIN, "cannot unmount '%s'"), return (zfs_error_fmt(zhp->zfs_hdl, libzfs_err,
mountpoint)); dgettext(TEXT_DOMAIN, "cannot unmount '%s'"),
mountpoint));
} else {
return (-1);
}
} }
return (0); return (0);