Prevent accessing more arguments than provided.

BUG=390111 Change-Id: Iecc2ac2354a82cca937a823e5c588c662264e36c Reviewed-on: https://chromium-review.googlesource.com/213551Tested-by: Nicolas Capens <capn@chromium.org> Reviewed-by: Jamie Madill <jmadill@chromium.org>

Prevent accessing more arguments than provided.
ffd7387a · Nicolas Capens · Nicolas Capens · 39b43463 · ffd7387a
Commit ffd7387a authored Aug 21, 2014 by Nicolas Capens Committed by Nicolas Capens Aug 22, 2014
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

ParseContext.cpp src/compiler/translator/ParseContext.cpp +1 -1

No files found.
--- a/src/compiler/translator/ParseContext.cpp
+++ b/src/compiler/translator/ParseContext.cpp
@@ -1613,7 +1613,7 @@ TIntermTyped *TParseContext::addConstructor(TIntermNode *arguments, const TType 
        for (size_t i = 0; i < fields.size(); i++)
        {
-            if ((*args)[i]->getAsTyped()->getType() != *fields[i]->type())
+            if (i >= args->size() || (*args)[i]->getAsTyped()->getType() != *fields[i]->type())
            {
                error(line, "Structure constructor arguments do not match structure fields", "Error");
                recover();