Commit 5ece3cafbd88d4da5c734e1810c4a2e6474b57b2
Committed by
J. Bruce Fields
1 parent
b0b0c0a26e
Exists in
master
and in
4 other branches
nfsd41: modify the members value of nfsd4_op_flags
The members of nfsd4_op_flags, (ALLOWED_WITHOUT_FH | ALLOWED_ON_ABSENT_FS) equals to ALLOWED_AS_FIRST_OP, maybe that's not what we want. OP_PUTROOTFH with op_flags = ALLOWED_WITHOUT_FH | ALLOWED_ON_ABSENT_FS, can't appears as the first operation with out SEQUENCE ops. This patch modify the wrong value of ALLOWED_WITHOUT_FH etc which was introduced by f9bb94c4. Cc: stable@kernel.org Reviewed-by: Benny Halevy <bhalevy@panasas.com> Signed-off-by: Mi Jinlong <mijinlong@cn.fujitsu.com> Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Showing 1 changed file with 2 additions and 2 deletions Side-by-side Diff
fs/nfsd/nfs4proc.c
... | ... | @@ -984,8 +984,8 @@ |
984 | 984 | void *); |
985 | 985 | enum nfsd4_op_flags { |
986 | 986 | ALLOWED_WITHOUT_FH = 1 << 0, /* No current filehandle required */ |
987 | - ALLOWED_ON_ABSENT_FS = 2 << 0, /* ops processed on absent fs */ | |
988 | - ALLOWED_AS_FIRST_OP = 3 << 0, /* ops reqired first in compound */ | |
987 | + ALLOWED_ON_ABSENT_FS = 1 << 1, /* ops processed on absent fs */ | |
988 | + ALLOWED_AS_FIRST_OP = 1 << 2, /* ops reqired first in compound */ | |
989 | 989 | }; |
990 | 990 | |
991 | 991 | struct nfsd4_operation { |