From: adilger <adilger>
Date: Sat, 16 Nov 2002 08:27:40 +0000 (+0000)
Subject: Don't dereference last_xid and last_committed in a request we just freed.
X-Git-Tag: 0.5.17~2
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=4083d0333ad9b2dafd730548084e33bbad2ec46b;p=fs%2Flustre-release.git

Don't dereference last_xid and last_committed in a request we just freed.
---

diff --git a/lustre/ptlrpc/client.c b/lustre/ptlrpc/client.c
index 1425498..43d40f9 100644
--- a/lustre/ptlrpc/client.c
+++ b/lustre/ptlrpc/client.c
@@ -666,10 +666,8 @@ int ptlrpc_queue_wait(struct ptlrpc_request *req)
                 spin_unlock(&conn->c_lock);
                 ptlrpc_req_finished(req); /* Must be called unlocked. */
                 spin_lock(&conn->c_lock);
-        }
-
-        /* Replay-enabled imports return commit-status information. */
-        if (req->rq_import->imp_flags & IMP_REPLAYABLE) {
+        } else /* if (req->rq_import->imp_flags & IMP_REPLAYABLE) */ {
+                /* Replay-enabled imports return commit-status information. */
                 /* XXX this needs to be per-import, or multiple MDS services on
                  * XXX the same system are going to interfere messily with each
                  * XXX others' transno spaces.