Don't divide in the validation of DrawElements

BUG=angleproject:1671 Change-Id: I58dd30d0aaffd1a776aa14a04011cbdd72181bf2 Reviewed-on: https://chromium-review.googlesource.com/648356 Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Jamie Madill <jmadill@chromium.org>

Don't divide in the validation of DrawElements
487653b1 · Corentin Wallez · Commit Bot · 6ab833ad · 487653b1
Commit 487653b1 authored Sep 01, 2017 by Corentin Wallez Committed by Commit Bot Sep 01, 2017
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

validationES.cpp src/libANGLE/validationES.cpp +2 -2

No files found.
--- a/src/libANGLE/validationES.cpp
+++ b/src/libANGLE/validationES.cpp
@@ -2871,8 +2871,8 @@ bool ValidateDrawElementsCommon(ValidationContext *context,
                return false;
            }
-            ASSERT(typeSize > 0);
+            ASSERT(isPow2(typeSize) && typeSize > 0);
-            if (elementArrayBuffer->getSize() % typeSize != 0)
+            if ((elementArrayBuffer->getSize() & (typeSize - 1)) != 0)
            {
                ANGLE_VALIDATION_ERR(context, InvalidOperation(), MismatchedByteCountType);
                return false;