Merge pull request #11618 from webpack/bugfix/11615

sokra · web-flow · commit 1658e2fd58e4 · 2020-10-09T23:56:35.000+02:00
fixes crash in SideEffectsFlagPlugin
diff --git a/lib/optimize/SideEffectsFlagPlugin.js b/lib/optimize/SideEffectsFlagPlugin.js
@@ -129,8 +129,20 @@ class SideEffectsFlagPlugin {
 							case "ForStatement":
 								if (
 									!parser.isPure(statement.init, statement.range[0]) ||
-									!parser.isPure(statement.test, statement.init.range[1]) ||
-									!parser.isPure(statement.update, statement.test.range[1])
+									!parser.isPure(
+										statement.test,
+										statement.init
+											? statement.init.range[1]
+											: statement.range[0]
+									) ||
+									!parser.isPure(
+										statement.update,
+										statement.test
+											? statement.test.range[1]
+											: statement.init
+											? statement.init.range[1]
+											: statement.range[0]
+									)
 								) {
 									hasSideEffects = true;
 								}
diff --git a/test/cases/side-effects/empty-modules/pure.js b/test/cases/side-effects/empty-modules/pure.js
@@ -26,3 +26,6 @@ const class2 = class {
 if ("") {
 	console.log("test");
 }
+for (; false; ) {}
+for (var i = 0; false; ) {}
+while (false) {}
