Move the resource lock into the handles lock at destroy

 - In order to prevent for possible issues, the missing
   mutex lock of the common implementation part is added

Change-Id: I6e09cca64b9dc301efeb42002d20581818fcd4a0

diff --git a/src/lib/mm_resource_manager_priv.c b/src/lib/mm_resource_manager_priv.c
index 3115dbb..8c430d1 100644 (file)
--- a/src/lib/mm_resource_manager_priv.c
+++ b/src/lib/mm_resource_manager_priv.c
@@ -205,13 +205,13 @@ int _mm_resource_manager_destroy(mm_resource_manager_h rm)
        __mm_resource_handles_lock();
        MM_RESOURCE_MANAGER_CHECK(handle);
        g_ptr_array_remove_fast(handles, handle);
+       __mm_resources_lock(handle);
        __mm_resource_handles_unlock();
 
        id = handle->id;
 
        MM_RM_INFO("RM #%"PRIu64" will be destroyed", _mm_rm_hash64(id));
 
-       __mm_resources_lock(handle);
        ret = __dbus_destroy(handle);
        if (ret != MM_RESOURCE_MANAGER_ERROR_NONE)
                MM_RM_ERROR("Dbus destroy request failed 0x%x", ret);