]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
nfsd4: fix security flavor of NFSv4.0 callback
authorJ. Bruce Fields <bfields@redhat.com>
Thu, 16 Aug 2012 21:01:21 +0000 (17:01 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 14 Sep 2012 16:59:25 +0000 (09:59 -0700)
commit9abbc5a9d54be7dbcc920d079ae790db3520f337
treee8f94564b225b43943b6e4a2a60880cdf1ae25be
parentbdd78615ccef0cb43ab4214c9f315efdefedcd6e
nfsd4: fix security flavor of NFSv4.0 callback

commit 39307655a1effa8d913bba054c0e985bfaca808c upstream.

Commit d5497fc693a446ce9100fcf4117c3f795ddfd0d2 "nfsd4: move rq_flavor
into svc_cred" forgot to remove cl_flavor from the client, leaving two
places (cl_flavor and cl_cred.cr_flavor) for the flavor to be stored.
After that patch, the latter was the one that was updated, but the
former was the one that the callback used.

Symptoms were a long delay on utime().  This is because the utime()
generated a setattr which recalled a delegation, but the cb_recall was
ignored by the client because it had the wrong security flavor.

Tested-by: Jamie Heilman <jamie@audible.transient.net>
Reported-by: Jamie Heilman <jamie@audible.transient.net>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/nfsd/nfs4callback.c
fs/nfsd/state.h