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

… semicolon Fixes less#3698 less#3373
b-kelly · Mar 9, 2022 · 02560eb · 02560eb
1 parent 3f05b5c
commit 02560eb
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: foo;
+}
+.test-no-trailing-semicolon2 {
+ --value: foo;
+}
+.test-no-trailing-semicolon3 {
+ --value: foo;
+}
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: foo
+}
+.test-no-trailing-semicolon2 {--value: foo}
+.test-no-trailing-semicolon3 { --value: foo }