commit 9e2c72a36c6e9df2c00935832d060df27cbcc35b
author Mike Stroyan <stroyan@google.com> Mon May 01 11:10:07 2017 -0600
committer Mike Stroyan <stroyan@users.noreply.github.com> Wed May 03 08:55:08 2017 -0600
tree 0f2baa6a2da8a6f8fa336957ab53023a2b6fb7d4
parent 716d38d429b41bb5a39c5638fe491021f47d4492

layers: Special case threading and VK_NULL_HANDLE

If a parameter is VK_NULL_HANDLE, then it needs no use count.

Fixes #1712

layers/threading.h

diff --git a/layers/threading.h b/layers/threading.h
index 426f6c1..20d0297 100644
--- a/layers/threading.h
+++ b/layers/threading.h
@@ -77,6 +77,9 @@
     std::mutex counter_lock;
     std::condition_variable counter_condition;
     void startWrite(debug_report_data *report_data, T object) {
+        if (object == VK_NULL_HANDLE) {
+            return;
+        }
         bool skipCall = false;
         loader_platform_thread_id tid = loader_platform_get_thread_id();
         std::unique_lock<std::mutex> lock(counter_lock);
@@ -147,6 +150,9 @@
     }
 
     void finishWrite(T object) {
+        if (object == VK_NULL_HANDLE) {
+            return;
+        }
         // Object is no longer in use
         std::unique_lock<std::mutex> lock(counter_lock);
         uses[object].writer_count -= 1;
@@ -159,6 +165,9 @@
     }
 
     void startRead(debug_report_data *report_data, T object) {
+        if (object == VK_NULL_HANDLE) {
+            return;
+        }
         bool skipCall = false;
         loader_platform_thread_id tid = loader_platform_get_thread_id();
         std::unique_lock<std::mutex> lock(counter_lock);
@@ -193,6 +202,9 @@
         }
     }
     void finishRead(T object) {
+        if (object == VK_NULL_HANDLE) {
+            return;
+        }
         std::unique_lock<std::mutex> lock(counter_lock);
         uses[object].reader_count -= 1;
         if ((uses[object].reader_count == 0) && (uses[object].writer_count == 0)) {