fix: more namespace checks

lddubeau · Jul 4, 2018 · a1add21 · a1add21
1 parent 4a1c99f
commit a1add21
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/lib/saxes.js b/lib/saxes.js
@@ -877,6 +877,10 @@ class SAXParser {
           case XMLNS_NAMESPACE:
             if (local === "") {
               this.fail(`The default namespace may not be set to
+${XMLNS_NAMESPACE}.`);
+            }
+            else {
+              this.fail(`May not assign a prefix (even "xmlns") to the URI \
 ${XMLNS_NAMESPACE}.`);
             }
             break;
@@ -885,6 +889,9 @@ ${XMLNS_NAMESPACE}.`);
               this.fail(`The default namespace may not be set to
 ${XML_NAMESPACE}.`);
             }
+            else if (local !== "xml") {
+              this.fail("May not assign the xml namespace to another prefix.");
+            }
             break;
           default:
           }