parser should not crash if to many arguments are passed to an IIFE

sokra · sokra · commit 158f35bcf190 · 2014-05-27T11:52:48.000+02:00
fixes webpack#284
diff --git a/lib/Parser.js b/lib/Parser.js
@@ -597,7 +597,7 @@ Parser.prototype.walkExpression = function walkExpression(expression) {
 			}), function() {
 				args.forEach(function(arg, idx) {
 					if(!arg) return;
-					if(params[idx].type !== "Identifier") return;
+					if(!params[idx] || params[idx].type !== "Identifier") return;
 					this.scope.renames["$"+params[idx].name] = arg;
 				}, this);
 				if(functionExpression.body.type === "BlockStatement")
diff --git a/test/cases/parsing/renaming/index.js b/test/cases/parsing/renaming/index.js
@@ -55,6 +55,18 @@ it("should be able to rename stuff by IIFE call", function() {
 			require);
 });
 
+it("should accept less parameters in a IIFE call", function() {
+	(function(r, require) {
+		r("./file").should.be.eql("ok");
+		(typeof require).should.be.eql("undefined");
+	}(require));
+});
+
+it("should accept more parameters in a IIFE call", function() {
+	(function() {
+	}(require));
+});
+
 it("should be able to rename stuff by IIFE call", function() {
 	(function(_exports, _module, _define, _require) {
 		_define(function(R, E, M) {
