fix frequently segfault on clean shutdown

diff --git a/lib/base/ebase.cpp b/lib/base/ebase.cpp
index 08c15610d4cb38dce3a0e2fbbe98fd5578f59217..effd7250924a2ed224df80b1b82888b1ede0efab 100644 (file)
--- a/lib/base/ebase.cpp
+++ b/lib/base/ebase.cpp
@@ -116,6 +116,16 @@ void eTimer::addTimeOffset( int offset )
 // mainloop
 ePtrList<eMainloop> eMainloop::existing_loops;
 
+eMainloop::~eMainloop()
+{
+       existing_loops.remove(this);
+       pthread_mutex_destroy(&recalcLock);
+       for (std::map<int, eSocketNotifier*>::iterator it(notifiers.begin());it != notifiers.end();++it)
+               it->second->stop();
+       while(m_timer_list.begin() != m_timer_list.end())
+               m_timer_list.begin()->stop();
+}
+
 void eMainloop::addSocketNotifier(eSocketNotifier *sn)
 {
        int fd = sn->getFD();

diff --git a/lib/base/ebase.h b/lib/base/ebase.h
index bab1b5f372aa71a0771595590853763a69cba646..4378711a9a0126bf555c9cea31c8185d012a5f85 100644 (file)
--- a/lib/base/ebase.h
+++ b/lib/base/ebase.h
@@ -217,13 +217,7 @@ public:
                existing_loops.push_back(this);
                pthread_mutex_init(&recalcLock, 0);
        }
-       virtual ~eMainloop()
-       {
-               existing_loops.remove(this);
-               pthread_mutex_destroy(&recalcLock);
-               for (std::map<int, eSocketNotifier*>::iterator it(notifiers.begin());it != notifiers.end();++it)
-                       it->second->stop();
-       }
+       virtual ~eMainloop();
 
        int looplevel() { return loop_level; }
 

diff --git a/lib/dvb/dvb.h b/lib/dvb/dvb.h
index b23d047f58592838f89fdfcac1f2a5e29bbe85d3..e09f0faec16d55f3427cc1c8a329090aface4145 100644 (file)
--- a/lib/dvb/dvb.h
+++ b/lib/dvb/dvb.h
@@ -33,9 +33,12 @@ public:
        eDVBRegisteredFrontend(eDVBFrontend *fe, iDVBAdapter *adap)
                :disable(new eTimer(eApp)), m_adapter(adap), m_frontend(fe), m_inuse(0)
        {
-               disable = new eTimer(eApp);
                CONNECT(disable->timeout, eDVBRegisteredFrontend::closeFrontend);
        }
+       ~eDVBRegisteredFrontend()
+       {
+               delete disable;
+       }
        void dec_use()
        {
                if (!--m_inuse)