JS: limit recursion in string merges, from OSSFuzz

tdewolff · Feb 6, 2021 · 5715b04 · 5715b04
1 parent 29261c3
commit 5715b04
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/js/js.go b/js/js.go
@@ -767,7 +767,7 @@ func (m *jsMinifier) minifyBinaryExpr(expr *js.BinaryExpr) bool {
 		strings := []*js.LiteralExpr{lit}
 
 		left := expr
-		for {
+		for len(strings) < 20 { // limit recursion
 			if lit, ok := left.X.(*js.LiteralExpr); ok && lit.TokenType == js.StringToken {
 				n += len(lit.Data) - 2
 				strings = append(strings, lit)