From: Nick Holland <nick.holland@partner.samsung.com>
Date: Wed, 15 Feb 2017 10:25:23 +0000 (+0000)
Subject: [3.0] Delete ThreadLocalStorage on shutdown
X-Git-Tag: accepted/tizen/3.0/common/20170221.125146~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F21%2F114921%2F1;p=platform%2Fcore%2Fuifw%2Fdali-core.git

[3.0] Delete ThreadLocalStorage on shutdown

Currently DALi ThreadLocalStorage object is not deleted
on shutdown.

Reported by Valgrind / Valkyrie

Change-Id: I376f2a05c149b3ac0d81e07fa1052fd44ee6d8a7
---

diff --git a/dali/internal/common/core-impl.cpp b/dali/internal/common/core-impl.cpp
index fafe5b3..8f07f35 100644
--- a/dali/internal/common/core-impl.cpp
+++ b/dali/internal/common/core-impl.cpp
@@ -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);
 }