From aaed6eb3d066144c1e661cc004767afc356f6b37 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Fri, 21 Jan 2011 10:08:26 -0800 Subject: [PATCH] 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 --- src/msg/SimpleMessenger.cc | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/msg/SimpleMessenger.cc b/src/msg/SimpleMessenger.cc index 4601e92d0b5d4..db742404251ad 100644 --- 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; } -- 2.39.5