nfsd: don't take/put an extra reference when putting a file
authorJeff Layton <jlayton@kernel.org>
Wed, 18 Jan 2023 17:31:37 +0000 (12:31 -0500)
committerChuck Lever <chuck.lever@oracle.com>
Wed, 26 Apr 2023 13:04:59 +0000 (09:04 -0400)
The last thing that filp_close does is an fput, so don't bother taking
and putting the extra reference.

Signed-off-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/filecache.c

index 5034944..51e2947 100644 (file)
@@ -382,10 +382,8 @@ nfsd_file_free(struct nfsd_file *nf)
        if (nf->nf_mark)
                nfsd_file_mark_put(nf->nf_mark);
        if (nf->nf_file) {
-               get_file(nf->nf_file);
-               filp_close(nf->nf_file, NULL);
                nfsd_file_check_write_error(nf);
-               fput(nf->nf_file);
+               filp_close(nf->nf_file, NULL);
        }
 
        /*