From: Lukasz Oleksak <l.oleksak@samsung.com>
Date: Fri, 2 Jul 2021 16:01:26 +0000 (+0200)
Subject: [ATSPI][SVACE] Fixes use of v variable after it was freed by eldbus_service_interface... 
X-Git-Tag: dali_2.0.34~7^2
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git;a=commitdiff_plain;h=276f8a333691c728dd0714c35b1af9c0dcbd300d

[ATSPI][SVACE] Fixes use of v variable after it was freed by eldbus_service_interface_unregister()

Change-Id: I44d89ab5532243ba79d212f0cb6ddd13289b34a8
---

diff --git a/automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/dbus-wrapper.cpp b/automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/dbus-wrapper.cpp
index d59df19..9011f2f 100644
--- a/automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/dbus-wrapper.cpp
+++ b/automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/dbus-wrapper.cpp
@@ -666,9 +666,11 @@ struct DefaultDBusWrapper : public DBusWrapper {
       DBUS_DEBUG( "registering interface %p (%d)", v, fallback ? 1 : 0 );
       destructors.push_back([=]() {
         DBUS_DEBUG( "unregistering interface %p", v );
+        {
+          std::lock_guard< std::mutex > lock( globalEntriesMutex );
+          globalEntries.erase( v );
+        }
         eldbus_service_interface_unregister( v );
-        std::lock_guard< std::mutex > lock( globalEntriesMutex );
-        globalEntries.erase( v );
       });
     }
   }