From a1b8c48bfb5ccbb6760e0b34fac5d80768bb4dfe Mon Sep 17 00:00:00 2001
From: Chris Forbes <chrisforbes@google.com>
Date: Tue, 1 Aug 2017 17:19:52 -0700
Subject: layers: Don't do bogus transitions in invalidateCommandBuffers

Total nonsense like:

  RECORDING -> INVALID_INCOMPLETE -> INVALID_COMPLETE

was possible when there were two invalidations.
---
 layers/core_validation.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'layers')

diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp
index 172cdbf0..eaadcbf7 100644
--- a/layers/core_validation.cpp
+++ b/layers/core_validation.cpp
@@ -4117,7 +4117,7 @@ void invalidateCommandBuffers(const layer_data *dev_data, std::unordered_set<GLO
                     "Invalidating a command buffer that's currently being recorded: 0x%p.", cb_node->commandBuffer);
             cb_node->state = CB_INVALID_INCOMPLETE;
         }
-        else {
+        else if (cb_node->state == CB_RECORDED) {
             cb_node->state = CB_INVALID_COMPLETE;
         }
         cb_node->broken_bindings.push_back(obj);
-- 
cgit v1.2.3