Check for too many positional params, fix #180

sparkprime · sparkprime · commit 54e36bb49426 · 2016-05-17T20:56:11.000-04:00
diff --git a/core/vm.cpp b/core/vm.cpp
@@ -1139,6 +1139,12 @@ class Interpreter {
                                    << i;
                                 throw makeError(ast.location, ss.str());
                             }
+                            if (i >= func->params.size()) {
+                                std::stringstream ss;
+                                ss << "Too many args, function has " << func->params.size()
+                                   << " parameter(s)";
+                                throw makeError(ast.location, ss.str());
+                            }
                             name = func->params[i].id;
                         }
                         // Special case for builtin functions -- leave identifier blank for

Original file line number	Diff line number	Diff line change
`@@ -1139,6 +1139,12 @@ class Interpreter {`
`1139`	`1139`	`<< i;`
`1140`	`1140`	`throw makeError(ast.location, ss.str());`
`1141`	`1141`	`}`
	`1142`	`+ if (i >= func->params.size()) {`
	`1143`	`+ std::stringstream ss;`
	`1144`	`+ ss << "Too many args, function has " << func->params.size()`
	`1145`	`+ << " parameter(s)";`
	`1146`	`+ throw makeError(ast.location, ss.str());`
	`1147`	`+ }`
`1142`	`1148`	`name = func->params[i].id;`
`1143`	`1149`	`}`
`1144`	`1150`	`// Special case for builtin functions -- leave identifier blank for`