nfsd: SUPPATTR_EXCLCREAT must be encoded before SECURITY_LABEL.
authorKinglong Mee <kinglongmee@gmail.com>
Thu, 30 Jul 2015 13:54:26 +0000 (21:54 +0800)
committerJ. Bruce Fields <bfields@redhat.com>
Mon, 31 Aug 2015 20:16:39 +0000 (16:16 -0400)
The encode order should be as the bitmask defined order.

Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4xdr.c

index ea05dfb132495b7e87e5edcf777b75afc32bcdbc..565b69ca04cd879e46cee8979da7d382619de665 100644 (file)
@@ -2735,12 +2735,6 @@ out_acl:
                *p++ = cpu_to_be32(stat.blksize);
        }
 #endif /* CONFIG_NFSD_PNFS */
-       if (bmval2 & FATTR4_WORD2_SECURITY_LABEL) {
-               status = nfsd4_encode_security_label(xdr, rqstp, context,
-                                                               contextlen);
-               if (status)
-                       goto out;
-       }
        if (bmval2 & FATTR4_WORD2_SUPPATTR_EXCLCREAT) {
                p = xdr_reserve_space(xdr, 16);
                if (!p)
@@ -2751,6 +2745,13 @@ out_acl:
                *p++ = cpu_to_be32(NFSD_SUPPATTR_EXCLCREAT_WORD2);
        }
 
+       if (bmval2 & FATTR4_WORD2_SECURITY_LABEL) {
+               status = nfsd4_encode_security_label(xdr, rqstp, context,
+                                                               contextlen);
+               if (status)
+                       goto out;
+       }
+
        attrlen = htonl(xdr->buf->len - attrlen_offset - 4);
        write_bytes_to_xdr_buf(xdr->buf, attrlen_offset, &attrlen, 4);
        status = nfs_ok;