From: Gunnar Beutner <gunnar@beutner.name>
Date: Wed, 3 Feb 2016 11:56:54 +0000 (+0100)
Subject: Fix memory leak in Stream::Close
X-Git-Tag: v2.5.0~559
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=caf3380e73558c7f67bb5deb0d1c88edcaad0900;p=icinga2

Fix memory leak in Stream::Close

refs #10655
---

diff --git a/lib/base/stream.cpp b/lib/base/stream.cpp
index 796a2c83d..240f9cb7b 100644
--- a/lib/base/stream.cpp
+++ b/lib/base/stream.cpp
@@ -76,9 +76,16 @@ bool Stream::WaitForData(int timeout)
 	return IsDataAvailable() || IsEof();
 }
 
+static void StreamDummyCallback(void)
+{ }
+
 void Stream::Close(void)
 {
 	OnDataAvailable.disconnect_all_slots();
+
+	/* Force signals2 to remove the slots, see https://stackoverflow.com/questions/2049291/force-deletion-of-slot-in-boostsignals2
+	 * for details. */
+	OnDataAvailable.connect(boost::bind(&StreamDummyCallback));
 }
 
 StreamReadStatus Stream::ReadLine(String *line, StreamReadContext& context, bool may_wait)