Delete ThreadLocalStorage on shutdown 76/114876/2
authorNick Holland <nick.holland@partner.samsung.com>
Wed, 15 Feb 2017 10:25:23 +0000 (10:25 +0000)
committerNick Holland <nick.holland@partner.samsung.com>
Wed, 15 Feb 2017 11:18:34 +0000 (11:18 +0000)
Currently DALi ThreadLocalStorage object is not deleted
on shutdown.

Reported by Valgrind / Valkyrie

Change-Id: I376f2a05c149b3ac0d81e07fa1052fd44ee6d8a7

dali/internal/common/core-impl.cpp

index 0f1e821..35827ad 100644 (file)
@@ -191,6 +191,7 @@ Core::~Core()
   if( tls )
   {
     tls->Remove();
+    delete tls;
   }
 
   // Stop relayout requests being raised on stage destruction
@@ -448,7 +449,7 @@ RelayoutController& Core::GetRelayoutController()
 void Core::CreateThreadLocalStorage()
 {
   // a pointer to the ThreadLocalStorage object will be stored in TLS
-  // and automatically deleted when the thread is killed
+  // The ThreadLocalStorage object should be deleted by the Core destructor
   new ThreadLocalStorage(this);
 }