Commit f7ee3b77 authored Aug 02, 2022 by Trond Myklebust Committed by Greg Kroah-Hartman Aug 25, 2022

NFSv4/pnfs: Fix a use-after-free bug in open

commit 2135e5d56278ffdb1c2e6d325dc6b87f669b9dac upstream.

If someone cancels the open RPC call, then we must not try to free
either the open slot or the layoutget operation arguments, since they
are likely still in use by the hung RPC call.

Fixes: 6949493884fe ("NFSv4: Don't hold the layoutget locks across multiple RPC calls")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 14b5a92e

fs/nfs/nfs4proc.c

+6 −5

Original line number	Diff line number	Diff line
		@@ -3039,12 +3039,13 @@ static int _nfs4_open_and_get_state(struct nfs4_opendata *opendata,
		}

		out:
		if (!opendata->cancelled) {
		if (opendata->lgp) {
		nfs4_lgopen_release(opendata->lgp);
		opendata->lgp = NULL;
		}
		if (!opendata->cancelled)
		nfs4_sequence_free_slot(&opendata->o_res.seq_res);
		}
		return ret;
		}