nfsd: drop the nfsd_put helper
authorJeff Layton <jlayton@kernel.org>
Wed, 3 Jan 2024 13:36:52 +0000 (08:36 -0500)
committerChuck Lever <chuck.lever@oracle.com>
Fri, 5 Jan 2024 03:52:27 +0000 (22:52 -0500)
commit64e6304169f1e1f078e7f0798033f80a7fb0ea46
tree80e43d184be307d43014ef9dcd54338d524cf762
parentbd018b98ba84ca0c80abac1ef23ce726a809e58c
nfsd: drop the nfsd_put helper

It's not safe to call nfsd_put once nfsd_last_thread has been called, as
that function will zero out the nn->nfsd_serv pointer.

Drop the nfsd_put helper altogether and open-code the svc_put in its
callers instead. That allows us to not be reliant on the value of that
pointer when handling an error.

Fixes: 2a501f55cd64 ("nfsd: call nfsd_last_thread() before final nfsd_put()")
Reported-by: Zhi Li <yieli@redhat.com>
Cc: NeilBrown <neilb@suse.de>
Signed-off-by: Jeffrey Layton <jlayton@redhat.com>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfsctl.c
fs/nfsd/nfsd.h