From 62e2a2881f6b441c136fb4ccb66ab491a5e6101f Mon Sep 17 00:00:00 2001 From: youzhongyang Date: Tue, 20 Sep 2022 18:25:21 -0400 Subject: [PATCH] Fix minor issues in namespace delegation support get_user_ns() is only done once for each namespace, so put_user_ns() should be done once too. Fix two typos in user_namespace/user_namespace_002.ksh and user_namespace/user_namespace_003.ksh. Reviewed-by: Richard Yao Reviewed-by: Brian Behlendorf Reviewed-by: Ryan Moeller Signed-off-by: Youzhong Yang Closes #13918 --- module/os/linux/spl/spl-zone.c | 2 +- .../tests/functional/user_namespace/user_namespace_002.ksh | 2 +- .../tests/functional/user_namespace/user_namespace_003.ksh | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/module/os/linux/spl/spl-zone.c b/module/os/linux/spl/spl-zone.c index b8a8b7cd8c..234ae7f6cd 100644 --- a/module/os/linux/spl/spl-zone.c +++ b/module/os/linux/spl/spl-zone.c @@ -415,8 +415,8 @@ spl_zone_fini(void) zone_dataset_t, zd_list); list_del(&zd->zd_list); kmem_free(zd, sizeof (*zd) + zd->zd_dsnamelen + 1); - put_user_ns(zds->zds_userns); } + put_user_ns(zds->zds_userns); list_del(&zds->zds_list); kmem_free(zds, sizeof (*zds)); } diff --git a/tests/zfs-tests/tests/functional/user_namespace/user_namespace_002.ksh b/tests/zfs-tests/tests/functional/user_namespace/user_namespace_002.ksh index b04898fa81..cfc478cd35 100755 --- a/tests/zfs-tests/tests/functional/user_namespace/user_namespace_002.ksh +++ b/tests/zfs-tests/tests/functional/user_namespace/user_namespace_002.ksh @@ -85,7 +85,7 @@ fi list="$($NSENTER zfs list -r -H -o name | tr '\n' ' ')" log_must test -z "$list" log_must zfs zone $proc_ns $TESTPOOL/userns -proc_ns_added="$ns" +proc_ns_added="$proc_ns" # 2. 'zfs list' list="$($NSENTER zfs list -r -H -o name $TESTPOOL | tr '\n' ' ')" diff --git a/tests/zfs-tests/tests/functional/user_namespace/user_namespace_003.ksh b/tests/zfs-tests/tests/functional/user_namespace/user_namespace_003.ksh index 2a875d09b6..6a746c6d33 100755 --- a/tests/zfs-tests/tests/functional/user_namespace/user_namespace_003.ksh +++ b/tests/zfs-tests/tests/functional/user_namespace/user_namespace_003.ksh @@ -88,7 +88,7 @@ list="$($NSENTER zfs list -r -H -o name | tr '\n' ' ')" log_must test -z "$list" log_must zfs zone $proc_ns $TESTPOOL/userns log_must zfs zone $proc_ns $TESTPOOL/otheruserns -proc_ns_added="$ns" +proc_ns_added="$proc_ns" # 2. 'zfs list' list="$($NSENTER zfs list -r -H -o name $TESTPOOL | tr '\n' ' ')"