diff options
| author | Jeff Layton <jlayton@kernel.org> | 2023-05-17 18:26:44 +0200 |
|---|---|---|
| committer | Chuck Lever <chuck.lever@oracle.com> | 2023-05-23 15:46:26 +0200 |
| commit | d53d70084d27f56bcdf5074328f2c9ec861be596 (patch) | |
| tree | fb02ea1cb60c5bb716fcdd0fea3bac06b9c3cceb /include | |
| parent | NFSD: Remove open coding of string copy (diff) | |
| download | linux-d53d70084d27f56bcdf5074328f2c9ec861be596.tar.xz linux-d53d70084d27f56bcdf5074328f2c9ec861be596.zip | |
nfsd: make a copy of struct iattr before calling notify_change
notify_change can modify the iattr structure. In particular it can
end up setting ATTR_MODE when ATTR_KILL_SUID is already set, causing
a BUG() if the same iattr is passed to notify_change more than once.
Make a copy of the struct iattr before calling notify_change.
Reported-by: Zhi Li <yieli@redhat.com>
Link: https://bugzilla.redhat.com/show_bug.cgi?id=2207969
Tested-by: Zhi Li <yieli@redhat.com>
Fixes: 34b91dda7124 ("NFSD: Make nfsd4_setattr() wait before returning NFS4ERR_DELAY")
Signed-off-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions