Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 19 Sep 2024 19:49:48 GMT
From:      Jamie Gritton <jamie@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: ae1a0648b05a - main - jail: unbreak the zfs.mount_snapshot option
Message-ID:  <202409191949.48JJnmdK083531@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by jamie:

URL: https://cgit.FreeBSD.org/src/commit/?id=ae1a0648b05acf798816e7b83b3c10856de5c8e5

commit ae1a0648b05acf798816e7b83b3c10856de5c8e5
Author:     Jamie Gritton <jamie@FreeBSD.org>
AuthorDate: 2024-09-19 19:49:01 +0000
Commit:     Jamie Gritton <jamie@FreeBSD.org>
CommitDate: 2024-09-19 19:49:01 +0000

    jail: unbreak the zfs.mount_snapshot option
    
    The change of its description from integer to boolean didn't actually
    change it to a boolean, but only made it impossible to set as either
    a boolean or an integer.
    
    Rather than make it work as a boolean parameter should, just revert
    to the old (working) integer parameter, and change the documentation
    to match.
    
    PR:             274263
    Reported by:    andrew.hotlab at hotmail
---
 sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vfsops.c | 2 +-
 usr.sbin/jail/jail.8                                   | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vfsops.c b/sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vfsops.c
index 23dc03cdac92..b96393df4626 100644
--- a/sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vfsops.c
+++ b/sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vfsops.c
@@ -100,7 +100,7 @@ static struct zfs_jailparam zfs_jailparam0 = {
 static int zfs_jailparam_slot;
 
 SYSCTL_JAIL_PARAM_SYS_NODE(zfs, CTLFLAG_RW, "Jail ZFS parameters");
-SYSCTL_JAIL_PARAM(_zfs, mount_snapshot, CTLTYPE_INT | CTLFLAG_RW, "B",
+SYSCTL_JAIL_PARAM(_zfs, mount_snapshot, CTLTYPE_INT | CTLFLAG_RW, "I",
 	"Allow mounting snapshots in the .zfs directory for unjailed datasets");
 
 SYSCTL_NODE(_vfs_zfs, OID_AUTO, version, CTLFLAG_RD, 0, "ZFS versions");
diff --git a/usr.sbin/jail/jail.8 b/usr.sbin/jail/jail.8
index 19e89ce661a9..881d07b1b626 100644
--- a/usr.sbin/jail/jail.8
+++ b/usr.sbin/jail/jail.8
@@ -23,7 +23,7 @@
 .\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
 .\" SUCH DAMAGE.
 .\"
-.Dd August 12, 2024
+.Dd September 19, 2024
 .Dt JAIL 8
 .Os
 .Sh NAME
@@ -789,8 +789,8 @@ Allow access to SYSV IPC semaphore and shared memory primitives, in the
 same manner as
 .Va sysvmsg .
 .It Va zfs.mount_snapshot
-Allow jailed users to access the contents of ZFS snapshots under the
-filesystem's
+When set to 1, jailed users may access the contents of ZFS snapshots
+under the filesystem's
 .Pa .zfs
 directory.
 If



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