commit 20bb8fbc02d5efb1a433efc2f170e0b267bd0cd0
author Józef Kucia <joseph.kucia@gmail.com> Fri Sep 23 12:45:04 2016 +0200
committer Tobin Ehlis <tobine@google.com> Fri Sep 23 10:26:32 2016 -0600
tree 9f8c23896198212ea1b4b8671c12f89b28ad9330
parent fa2f36cc8a6e72bd8bd00c8fd27d49f56a4c3331

layers: Prevent reading uninitialized "count" when validating array parameters

layers/parameter_validation_utils.h

diff --git a/layers/parameter_validation_utils.h b/layers/parameter_validation_utils.h
index a1d2e1f..5084d5a 100644
--- a/layers/parameter_validation_utils.h
+++ b/layers/parameter_validation_utils.h
@@ -174,7 +174,7 @@
     bool skip_call = false;
 
     // Count parameters not tagged as optional cannot be 0
-    if ((count == 0) && countRequired) {
+    if (countRequired && (count == 0)) {
         skip_call |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0, __LINE__,
                             REQUIRED_PARAMETER, LayerName, "%s: parameter %s must be greater than 0", apiName,
                             countName.get_name().c_str());