From 5fdb8a4a70426041a5f8c961973242289ce69e0d Mon Sep 17 00:00:00 2001 From: Andrew Hanushevsky Date: Mon, 16 Sep 2019 17:13:32 -0700 Subject: [PATCH] [SSI] Make sure the request sessN pointer is always valid. --- src/XrdSsi/XrdSsiFileReq.cc | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/XrdSsi/XrdSsiFileReq.cc b/src/XrdSsi/XrdSsiFileReq.cc index 454595ca955..da94f3bd8e2 100644 --- a/src/XrdSsi/XrdSsiFileReq.cc +++ b/src/XrdSsi/XrdSsiFileReq.cc @@ -487,6 +487,7 @@ void XrdSsiFileReq::Finalize() mHelper.UnLock(); wt4fin.Wait(); } + sessN = "n/a"; return; // Request is bound so we can finish right off. @@ -499,13 +500,15 @@ void XrdSsiFileReq::Finalize() Stats.Bump(Stats.ReqFinished); if (cancel) Stats.Bump(Stats.ReqCancels); Finished(cancel); // This object may be deleted! + sessN = "n/a"; return; break; // The following two cases may happen but it's safe to ignore them. // case isAbort: - case isDone: return; + case isDone: sessN = "bad"; + return; break; default: break; }