Commit f7c45deb authored by J. Bruce Fields's avatar J. Bruce Fields Committed by Kleber Sacilotto de Souza

nfsd: fix corrupted reply to badly ordered compound

BugLink: https://bugs.launchpad.net/bugs/1798770

[ Upstream commit 5b7b15ae ]

We're encoding a single op in the reply but leaving the number of ops
zero, so the reply makes no sense.

Somewhat academic as this isn't a case any real client will hit, though
in theory perhaps that could change in a future protocol extension.
Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
Signed-off-by: default avatarSasha Levin <alexander.levin@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarStefan Bader <stefan.bader@canonical.com>
Signed-off-by: default avatarKleber Sacilotto de Souza <kleber.souza@canonical.com>
parent a28ff3c4
...@@ -1632,6 +1632,7 @@ nfsd4_proc_compound(struct svc_rqst *rqstp, ...@@ -1632,6 +1632,7 @@ nfsd4_proc_compound(struct svc_rqst *rqstp,
if (status) { if (status) {
op = &args->ops[0]; op = &args->ops[0];
op->status = status; op->status = status;
resp->opcnt = 1;
goto encode_op; goto encode_op;
} }
......
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