diff options
| author | Józef Kucia <joseph.kucia@gmail.com> | 2016-09-23 12:45:04 +0200 |
|---|---|---|
| committer | Tobin Ehlis <tobine@google.com> | 2016-09-23 10:26:32 -0600 |
| commit | 6939d202c3b339fcf75e0be66e3e0091a55c121c (patch) | |
| tree | 3121f165b8c5768535d24bf22fed837613ce6c74 /layers/parameter_validation_utils.h | |
| parent | b01b70a8b7e3e5420e2822e6f9115b0c0f5eccdc (diff) | |
| download | usermoji-6939d202c3b339fcf75e0be66e3e0091a55c121c.tar.xz | |
layers: Prevent reading uninitialized "count" when validating array parameters
Diffstat (limited to 'layers/parameter_validation_utils.h')
| -rw-r--r-- | layers/parameter_validation_utils.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/layers/parameter_validation_utils.h b/layers/parameter_validation_utils.h index a1d2e1f6..5084d5a1 100644 --- a/layers/parameter_validation_utils.h +++ b/layers/parameter_validation_utils.h @@ -174,7 +174,7 @@ bool validate_array(debug_report_data *report_data, const char *apiName, const P 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()); |