fix(parser): fix crash when css variable property does not end with a…

… semicolon Fixes less#3698 less#3373
b-kelly · Mar 9, 2022 · 1e62fae · 1e62fae
1 parent 3f05b5c
commit 1e62fae
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 2 deletions.
diff --git a/packages/less/src/less/parser/parser.js b/packages/less/src/less/parser/parser.js
@@ -1541,7 +1541,7 @@ const Parser = function Parser(context, imports, fileInfo) {
 
                         // Custom property values get permissive parsing
                         if (name[0].value && name[0].value.slice(0, 2) === '--') {
-                            value = this.permissiveValue();
+                            value = this.permissiveValue(/[;}]/);
                         }
                         // Try to store values as anonymous
                         // If we need the value later we'll re-parse it in ruleset.parseValue

diff --git a/packages/test-data/css/_main/permissive-parse.css b/packages/test-data/css/_main/permissive-parse.css
@@ -35,3 +35,12 @@ foo[attr="blah"] {
   --value: a /* { ; } */;
   --comment-within: ( /* okay?; comment; */ );
 }
+.test-no-trailing-semicolon {
+  --value: blue;
+}
+.test-no-trailing-semicolon2 {
+  --value: blue;
+}
+.test-no-trailing-semicolon3 {
+  --value: blue;
+}
diff --git a/packages/test-data/less/_main/permissive-parse.less b/packages/test-data/less/_main/permissive-parse.less
@@ -47,4 +47,9 @@
 .test-rule-comment {
   --value: a/* { ; } */;
   --comment-within: ( /* okay?; comment; */ );
-}
+}
+.test-no-trailing-semicolon {
+  --value: blue
+}
+.test-no-trailing-semicolon2 {--value: blue}
+.test-no-trailing-semicolon3 { --value: blue }