On Sat, 30 Jul 2022, Chuck Lever III wrote: > > @@ -837,10 +821,8 @@ nfsd4_create(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, > > > > if (attrs.label_failed) > > create->cr_bmval[2] &= ~FATTR4_WORD2_SECURITY_LABEL; > > - > > - if (create->cr_acl != NULL) > > - do_set_nfs4_acl(rqstp, &resfh, create->cr_acl, > > - create->cr_bmval); > > + if (attrs.label_failed) > > + create->cr_bmval[0] &= ~FATTR4_WORD0_ACL; > > I think this needs to be "if (attrs.acl_failed)". I've fixed this > in my tree. Yes, of course :-(. Thanks, NeilBrown