osd/PG: check the connection first in fulfill_log

          No need to prepare reply message if the connection not exist

  Signed-off-by: huangjun <hjwsm1989@gmail.com>

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index 312d12b685fdb448eac5176f84bed0d338358992..1cba004bbe2a8094110e273f1de056c4f6c9a778 100644 (file)
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -4815,6 +4815,9 @@ void PG::fulfill_log(
   dout(10) << "log request from " << from << dendl;
   assert(from == primary);
   assert(query.type != pg_query_t::INFO);
+  ConnectionRef con = osd->get_con_osd_cluster(
+    from.osd, get_osdmap()->get_epoch());
+  if (!con) return;
 
   MOSDPGLog *mlog = new MOSDPGLog(
     from.shard, pg_whoami.shard,
@@ -4841,14 +4844,8 @@ void PG::fulfill_log(
 
   dout(10) << " sending " << mlog->log << " " << mlog->missing << dendl;
 
-  ConnectionRef con = osd->get_con_osd_cluster(
-    from.osd, get_osdmap()->get_epoch());
-  if (con) {
-    osd->share_map_peer(from.osd, con.get(), get_osdmap());
-    osd->send_message_osd_cluster(mlog, con.get());
-  } else {
-    mlog->put();
-  }
+  osd->share_map_peer(from.osd, con.get(), get_osdmap());
+  osd->send_message_osd_cluster(mlog, con.get());
 }