From: Sage Weil <sage@inktank.com>
Date: Mon, 20 May 2013 20:34:05 +0000 (-0700)
Subject: mds: better error check on sessionmap load
X-Git-Tag: v0.64~94
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=d2a42538b1e349027e31bc2cb065e50af6294533;p=ceph.git

mds: better error check on sessionmap load

Would have helped with #5104.

Signed-off-by: Sage Weil <sage@inktank.com>
---

diff --git a/src/mds/SessionMap.cc b/src/mds/SessionMap.cc
index 53fe90c10ba0..f17fa9e46c10 100644
--- a/src/mds/SessionMap.cc
+++ b/src/mds/SessionMap.cc
@@ -18,6 +18,7 @@
 #include "osdc/Filer.h"
 
 #include "common/config.h"
+#include "common/errno.h"
 #include "include/assert.h"
 
 #define dout_subsys ceph_subsys_mds
@@ -77,6 +78,10 @@ void SessionMap::load(Context *onload)
 void SessionMap::_load_finish(int r, bufferlist &bl)
 { 
   bufferlist::iterator blp = bl.begin();
+  if (r < 0) {
+    derr << "_load_finish got " << cpp_strerror(r) << dendl;
+    assert(0 == "failed to load sessionmap");
+  }
   dump();
   decode(blp);  // note: this sets last_cap_renew = now()
   dout(10) << "_load_finish v " << version