msgr: always start reaper

If we didn't explicitly bind (i.e. are a client), then we don't start
the accepter.  That's fine. But the reaper thread start was also
conditional, when it shouldn't be; otherwise the client can't clean up
old Pipes (and their sockets).

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

diff --git a/src/msg/SimpleMessenger.cc b/src/msg/SimpleMessenger.cc
index 4601e92d0b5d4e4ab83eb95e94cbcdf58fe73d1f..db742404251adb2d654693421034e6946f0c1ded 100644 (file)
--- a/src/msg/SimpleMessenger.cc
+++ b/src/msg/SimpleMessenger.cc
@@ -2410,12 +2410,11 @@ int SimpleMessenger::start(bool nodaemon)
   }
 
   // go!
-  if (did_bind) {
+  if (did_bind)
     accepter.start();
 
-    reaper_started = true;
-    reaper_thread.create();
-  }
+  reaper_started = true;
+  reaper_thread.create();
   return 0;
 }