git: 6580d040861d - main - nfs_nfsdserv.c: Fix handling of archive flag for mkdir

The branch main has been updated by rmacklem:

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

commit 6580d040861dfbf6c630a93cbf41f2a2c7e7b327
Author:     Rick Macklem <rmacklem@FreeBSD.org>
AuthorDate: 2026-03-16 23:22:37 +0000
Commit:     Rick Macklem <rmacklem@FreeBSD.org>
CommitDate: 2026-03-16 23:22:37 +0000

    nfs_nfsdserv.c: Fix handling of archive flag for mkdir
    
    An NFSv4.1/4.2 client can set/clear the archive, hidden
    and system flags when creating non-regular files, such
    as directories.
    
    Without this patch, the setting of va_flags causes an
    EPERM failure, since they are specified for VOP_MKDIR(),
    VOP_MKNOD() and VOP_SYMLINK().
    
    This patch sets va_flags == VNOVAL for the above VOP_xxx()
    calls and then sets/clears the flags after creation,
    which fixes the problem.
    
    This bug only affects the Windows NFSv4.1/4.2 client.
    
    PR:     293691
    Tested by:      Dan Shelton <dan.f.shelton@gmail.com>
    MFC after:      2 weeks
---
 sys/fs/nfsserver/nfs_nfsdserv.c | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/sys/fs/nfsserver/nfs_nfsdserv.c b/sys/fs/nfsserver/nfs_nfsdserv.c
index b5f5b9bec9fc..b5c7f4743be0 100644
--- a/sys/fs/nfsserver/nfs_nfsdserv.c
+++ b/sys/fs/nfsserver/nfs_nfsdserv.c
@@ -1438,7 +1438,7 @@ nfsrvd_mknod(struct nfsrv_descript *nd, __unused int isdgram,
 	vnode_t vp, dirp = NULL;
 	nfsattrbit_t attrbits;
 	char *bufp = NULL, *pathcp = NULL;
-	u_long *hashp, cnflags;
+	u_long *hashp, cnflags, setflags;
 	NFSACL_T *aclp = NULL, *daclp = NULL;
 	struct thread *p = curthread;
 
@@ -1605,9 +1605,13 @@ nfsrvd_mknod(struct nfsrv_descript *nd, __unused int isdgram,
 		}
 	}
 
+	/* For NFSv4, set na_flags via nfsrv_fixattr(). */
+	setflags = nva.na_flags;
+	nva.na_flags = VNOVAL;
 	nd->nd_repstat = nfsvno_mknod(&named, &nva, nd->nd_cred, p);
 	if (!nd->nd_repstat) {
 		vp = named.ni_vp;
+		nva.na_flags = setflags;
 		nfsrv_fixattr(nd, vp, &nva, aclp, daclp, p, &attrbits, false);
 		nd->nd_repstat = nfsvno_getfh(vp, fhp, p);
 		if ((nd->nd_flag & ND_NFSV3) && !nd->nd_repstat)
@@ -2115,10 +2119,14 @@ nfsrvd_symlinksub(struct nfsrv_descript *nd, struct nameidata *ndp,
     int pathlen)
 {
 	u_int32_t *tl;
+	u_long setflags;
 
+	setflags = nvap->na_flags;
+	nvap->na_flags = (u_long)VNOVAL;
 	nd->nd_repstat = nfsvno_symlink(ndp, nvap, pathcp, pathlen,
 	    !(nd->nd_flag & ND_NFSV2), nd->nd_saveduid, nd->nd_cred, p, exp);
 	if (!nd->nd_repstat && !(nd->nd_flag & ND_NFSV2)) {
+		nvap->na_flags = setflags;
 		nfsrv_fixattr(nd, ndp->ni_vp, nvap, aclp, NULL, p, attrbitp,
 		    false);
 		if (nd->nd_flag & ND_NFSV3) {
@@ -2249,12 +2257,16 @@ nfsrvd_mkdirsub(struct nfsrv_descript *nd, struct nameidata *ndp,
 {
 	vnode_t vp;
 	u_int32_t *tl;
+	u_long setflags;
 
+	setflags = nvap->na_flags;
+	nvap->na_flags = (u_long)VNOVAL;
 	NFSVNO_SETATTRVAL(nvap, type, VDIR);
 	nd->nd_repstat = nfsvno_mkdir(ndp, nvap, nd->nd_saveduid,
 	    nd->nd_cred, p, exp);
 	if (!nd->nd_repstat) {
 		vp = ndp->ni_vp;
+		nvap->na_flags = setflags;
 		nfsrv_fixattr(nd, vp, nvap, aclp, daclp, p, attrbitp, false);
 		nd->nd_repstat = nfsvno_getfh(vp, fhp, p);
 		if (!(nd->nd_flag & ND_NFSV4) && !nd->nd_repstat)