commit 37f9f6af59d06f5e4fc26e4ddd039e64f4764d3d
author Tony Barbour <tony@LunarG.com> Wed Apr 06 18:17:57 2016 -0600
committer Tony Barbour <tony@LunarG.com> Wed Apr 06 18:17:57 2016 -0600
tree a641ec414c9807357211fef4a44987e83332b6eb
parent f3f4ee9c8529ab122a1eaf972affa0063cbe82cb

layers: Move call to validateCommandBuffersNotInUse inside lock

Fixes segv in CTS multithreaded command pool test

Change-Id: I80da12fd2bb6856e3c6824f17dd92f1597121a54

layers/core_validation.cpp

diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp
index aa5eafb..7a2dd69 100644
--- a/layers/core_validation.cpp
+++ b/layers/core_validation.cpp
@@ -6243,9 +6243,12 @@
     }
     dev_data->commandPoolMap.erase(commandPool);
 
+
+    VkBool32 result = validateCommandBuffersNotInUse(dev_data, commandPool);
+
     loader_platform_thread_unlock_mutex(&globalLock);
 
-    if (VK_TRUE == validateCommandBuffersNotInUse(dev_data, commandPool))
+    if (result)
         return;
 
     if (!skipCall)