From: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Date: Tue, 28 May 2013 10:55:19 +0000 (+0200)
Subject: mds/Server.cc: fix explicit null dereferenced
X-Git-Tag: v0.65~168^2~17
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=39a53e5c72c0c02165cbb282bf940f48ae317d6a;p=ceph.git

mds/Server.cc: fix explicit null dereferenced

CID 716928 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)
  var_deref_model: Passing null pointer "session" to function
  "Session::trim_completed_requests(tid_t)", which dereferences it.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
---

diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index 98dafc3e285b..644b3ade0052 100644
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -1115,6 +1115,7 @@ void Server::handle_client_request(MClientRequest *req)
   // trim completed_request list
   if (req->get_oldest_client_tid() > 0) {
     dout(15) << " oldest_client_tid=" << req->get_oldest_client_tid() << dendl;
+    assert(session);
     session->trim_completed_requests(req->get_oldest_client_tid());
   }