objecter: fix use-after-free

messenger consumes the m reference.  Yay valgrind.

Signed-off-by: Sage Weil <sage@newdream.net>

diff --git a/src/osdc/Objecter.cc b/src/osdc/Objecter.cc
index 07dc34b52b1645919a05fdfa33a7429a8e7252da..04816aba00012a17f89583c7838580c8649c0169 100644 (file)
--- a/src/osdc/Objecter.cc
+++ b/src/osdc/Objecter.cc
@@ -1068,10 +1068,10 @@ void Objecter::send_op(Op *op)
   if (op->priority)
     m->set_priority(op->priority);
 
-  messenger->send_message(m, op->session->con);
-
   logger->inc(l_osdc_op_send);
   logger->inc(l_osdc_op_send_bytes, m->get_data().length());
+
+  messenger->send_message(m, op->session->con);
 }
 
 int Objecter::calc_op_budget(Op *op)