[3.0] Delete ThreadLocalStorage on shutdown 21/114921/1
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 14:35:36 +0000 (14:35 +0000)
Currently DALi ThreadLocalStorage object is not deleted
on shutdown.

Reported by Valgrind / Valkyrie

Change-Id: I376f2a05c149b3ac0d81e07fa1052fd44ee6d8a7

dali/internal/common/core-impl.cpp

index fafe5b3..8f07f35 100644 (file)
@@ -192,6 +192,7 @@ Core::~Core()
   if( tls )
   {
     tls->Remove();
+    delete tls;
   }
 
   // Stop relayout requests being raised on stage destruction
@@ -464,7 +465,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);
 }