fix: raise error on character reference outside CHAR production

lddubeau · Jul 4, 2018 · 30fb540 · 30fb540
1 parent 04e1593
commit 30fb540
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/saxes.js b/lib/saxes.js
@@ -1036,7 +1036,7 @@ class SAXParser {
       }
     }
 
-    if (Number.isNaN(num)) {
+    if (Number.isNaN(num) || num > 0x10FFFF) {
       this.fail("Invalid character entity");
       return `&${this.entity};`;
     }

diff --git a/test/conformance.js b/test/conformance.js
@@ -29,7 +29,8 @@ class SaxesDriver extends BaseDriver {
           errors.push(err);
         };
 
-        parser.close(source);
+        parser.write(source);
+        parser.end();
         this.processResult(test, handling, errors.length === 0);
       });
   }