Make sure all threads are dead before calling fork()

fixes #7653

lib/base/application.cpp

@@ -109,7 +109,7 @@ void Application::Exit(int rc)
 		logger->Flush();
 	}
 
-	UninitializeBase();
+	Timer::Uninitialize();
 
 #ifdef _DEBUG
 	exit(rc);
@@ -120,8 +120,6 @@ void Application::Exit(int rc)
 
 void Application::InitializeBase(void)
 {
-	Timer::Initialize();
-
 #ifndef _WIN32
 	rlimit rl;
 	if (getrlimit(RLIMIT_NOFILE, &rl) >= 0) {
@@ -154,11 +152,6 @@ void Application::InitializeBase(void)
 	Utility::ExecuteDeferredInitializers();
 }
 
-void Application::UninitializeBase(void)
-{
-	Timer::Uninitialize();
-}
-
 /**
  * Retrieves a pointer to the application singleton object.
  *

lib/base/application.hpp

@@ -43,7 +43,6 @@ public:
 	~Application(void);
 
 	static void InitializeBase(void);
-	static void UninitializeBase(void);
 
 	static Application::Ptr GetInstance(void);
 

lib/base/timer.cpp

@@ -81,7 +81,8 @@ void Timer::Uninitialize(void)
 		l_CV.notify_all();
 	}
 
-	l_Thread.join();
+	if (l_Thread.joinable())
+		l_Thread.join();
 }
 
 /**

test/test.cpp

@@ -21,6 +21,7 @@
 #define BOOST_TEST_MODULE icinga2_test
 
 #include "base/application.hpp"
+#include "base/timer.hpp"
 #include <BoostTestTargetConfig.h>
 
 using namespace icinga;
@@ -30,11 +31,12 @@ struct InitLibBase
 	InitLibBase(void)
 	{
 		Application::InitializeBase();
+		Timer::Initialize();
 	}
 
 	~InitLibBase(void)
 	{
-		Application::UninitializeBase();
+		Timer::Uninitialize();
 	}
 };