commit 487653b1a6b7f0b46b427186fa11c6379ec70f49
author Corentin Wallez <cwallez@chromium.org> Fri Sep 01 17:17:55 2017 -0400
committer Commit Bot <commit-bot@chromium.org> Fri Sep 01 22:41:11 2017 +0000
tree ec631b24654350f78b15f73a6d86385c2b0463f1
parent 6ab833add9a3b1c762bff1c38ecf1f30e55f5b54

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>

src/libANGLE/validationES.cpp

diff --git a/src/libANGLE/validationES.cpp b/src/libANGLE/validationES.cpp
index 7b55135..ad66ede 100644
--- a/src/libANGLE/validationES.cpp
+++ b/src/libANGLE/validationES.cpp
@@ -2871,8 +2871,8 @@
                 return false;
             }
 
-            ASSERT(typeSize > 0);
-            if (elementArrayBuffer->getSize() % typeSize != 0)
+            ASSERT(isPow2(typeSize) && typeSize > 0);
+            if ((elementArrayBuffer->getSize() & (typeSize - 1)) != 0)
             {
                 ANGLE_VALIDATION_ERR(context, InvalidOperation(), MismatchedByteCountType);
                 return false;