tyler.durden/icinga2
1
0
Fork 0
mirror of https://github.com/Icinga/icinga2.git synced 2025-07-25 22:54:57 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix for double-free (and possibly other memory-corruption related) crashes at logrotate time

Browse Source 
this is a direct fix of the issue revealing the problem that leads to crash

verification done with a patched icinga2 where the execution-order of the code lines of counter-parts involved in re-incrementing/decrementing Timer:Ptr is forced to be the one that leads to the obeserverd segfaults

refs #6737
This commit is contained in:
Elias Ohm 2019-04-17 22:31:42 +02:00 committed by Michael Friedrich
parent d0e94c8efc
commit 52e3db279a
3 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/base/timer.cpp
View File

@ -110,7 +110,7 @@ void Timer::UninitializeThread()
void Timer::Call() void Timer::Call()
{ {
try { try {
OnTimerExpired(Timer::Ptr(this)); OnTimerExpired(this);
} catch (...) { } catch (...) {
InternalReschedule(true); InternalReschedule(true);
@ -301,8 +301,6 @@ void Timer::TimerThreadProc()
continue; continue;
} }
Timer::Ptr ptimer = timer;
/* Remove the timer from the list so it doesn't get called again /* Remove the timer from the list so it doesn't get called again
* until the current call is completed. */ * until the current call is completed. */
l_Timers.erase(timer); l_Timers.erase(timer);
@ -312,6 +310,6 @@ void Timer::TimerThreadProc()
lock.unlock(); lock.unlock();
/* Asynchronously call the timer. */ /* Asynchronously call the timer. */
Utility::QueueAsyncCallback(std::bind(&Timer::Call, ptimer)); Utility::QueueAsyncCallback(std::bind(&Timer::Call, timer));
} }
} }

2
lib/base/timer.hpp
View File

@ -39,7 +39,7 @@ public:
void Reschedule(double next = -1); void Reschedule(double next = -1);
double GetNext() const; double GetNext() const;
boost::signals2::signal<void(const Timer::Ptr&)> OnTimerExpired; boost::signals2::signal<void(const Timer * const&)> OnTimerExpired;
private: private:
double m_Interval{0}; /**< The interval of the timer. */ double m_Interval{0}; /**< The interval of the timer. */

2
test/base-timer.cpp
View File

@ -24,7 +24,7 @@ BOOST_AUTO_TEST_CASE(interval)
int counter = 0; int counter = 0;
static void Callback(const Timer::Ptr&) static void Callback(const Timer * const&)
{ {
counter++; counter++;
} }