Commit bc1b5acb authored by Mahmoud Adam's avatar Mahmoud Adam Committed by Chuck Lever

nfsd: fix file memleak on client_opens_release

seq_release should be called to free the allocated seq_file

Cc: stable@vger.kernel.org # v5.3+
Signed-off-by: default avatarMahmoud Adam <mngyadam@amazon.com>
Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
Fixes: 78599c42 ("nfsd4: add file to display list of client's opens")
Reviewed-by: default avatarNeilBrown <neilb@suse.de>
Tested-by: default avatarJeff Layton <jlayton@kernel.org>
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
parent 3fd2ca5b
...@@ -2804,7 +2804,7 @@ static int client_opens_release(struct inode *inode, struct file *file) ...@@ -2804,7 +2804,7 @@ static int client_opens_release(struct inode *inode, struct file *file)
/* XXX: alternatively, we could get/drop in seq start/stop */ /* XXX: alternatively, we could get/drop in seq start/stop */
drop_client(clp); drop_client(clp);
return 0; return seq_release(inode, file);
} }
static const struct file_operations client_states_fops = { static const struct file_operations client_states_fops = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment