From d641bbe4fc1a81102456b16535309e90e1c10b53 Mon Sep 17 00:00:00 2001 From: "John L. Hammond" Date: Thu, 17 Nov 2016 11:39:19 -0600 Subject: [PATCH] LU-8811 mdc: avoid returning freed request In mdc_close() if ptlrpc_request_pack() fails then set req to NULL so that an already freed request is not returned in *request. Signed-off-by: John L. Hammond Change-Id: I7460c650b736f51d78981f8a1e50758f7acf7c77 Reviewed-on: https://review.whamcloud.com/23843 Reviewed-by: Patrick Farrell Reviewed-by: Andreas Dilger Tested-by: Jenkins Tested-by: Maloo --- lustre/mdc/mdc_request.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lustre/mdc/mdc_request.c b/lustre/mdc/mdc_request.c index 0372052..75355fd 100644 --- a/lustre/mdc/mdc_request.c +++ b/lustre/mdc/mdc_request.c @@ -797,6 +797,7 @@ static int mdc_close(struct obd_export *exp, struct md_op_data *op_data, rc = ptlrpc_request_pack(req, LUSTRE_MDS_VERSION, MDS_CLOSE); if (rc) { ptlrpc_request_free(req); + req = NULL; GOTO(out, rc); } -- 1.8.3.1