From: Josh Durgin <josh.durgin@dreamhost.com>
Date: Wed, 14 Dec 2011 02:08:43 +0000 (-0800)
Subject: OSDMonitor: remove useless check
X-Git-Tag: v0.40~96^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=abecbc5958306bc63b4512c642d4e24ba4aba40a;p=ceph.git

OSDMonitor: remove useless check

Session was already verified to exist before this.

Signed-off-by: Josh Durgin <josh.durgin@dreamhost.com>
---

diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc
index 5a7ec946d2dc..87e3f195453e 100644
--- a/src/mon/OSDMonitor.cc
+++ b/src/mon/OSDMonitor.cc
@@ -2160,10 +2160,9 @@ bool OSDMonitor::preprocess_pool_op_create(MPoolOp *m)
   }
   if ((m->auid && !session->caps.check_privileges(PAXOS_OSDMAP, MON_CAP_W, m->auid)) &&
       !session->caps.check_privileges(PAXOS_OSDMAP, MON_CAP_W)) {
-    if (session)
-      dout(5) << "attempt to create new pool without sufficient auid privileges!"
-	      << "message: " << *m  << std::endl
-	      << "caps: " << m->get_session()->caps << dendl;
+    dout(5) << "attempt to create new pool without sufficient auid privileges!"
+	    << "message: " << *m  << std::endl
+	    << "caps: " << session->caps << dendl;
     _pool_op_reply(m, -EPERM, osdmap.get_epoch());
     return true;
   }