(ws-axiom) 01/02: Properly validate prefixes passed to setPrefix

[veithen]

This is an automated email from the ASF dual-hosted git repository.

veithen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ws-axiom.git

commit 37fc15f5c0bfe0c1566ec3e823c4419f265b0a6f
Author: Andreas Veithen <andreas.veit...@gmail.com>
AuthorDate: Sun Dec 17 21:01:15 2023 +0000

    Properly validate prefixes passed to setPrefix
---
 .../org/apache/axiom/om/impl/dom/W3CDOMLevel2ImplementationTest.java    | 1 -
 .../java/org/apache/axiom/dom/impl/mixin/DOMNSAwareNamedNodeMixin.java  | 2 ++
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/implementations/axiom-dom/src/test/java/org/apache/axiom/om/impl/dom/W3CDOMLevel2ImplementationTest.java
 
b/implementations/axiom-dom/src/test/java/org/apache/axiom/om/impl/dom/W3CDOMLevel2ImplementationTest.java
index e74f23b50..3de70ed3a 100644
--- 
a/implementations/axiom-dom/src/test/java/org/apache/axiom/om/impl/dom/W3CDOMLevel2ImplementationTest.java
+++ 
b/implementations/axiom-dom/src/test/java/org/apache/axiom/om/impl/dom/W3CDOMLevel2ImplementationTest.java
@@ -53,7 +53,6 @@ public class W3CDOMLevel2ImplementationTest extends TestCase {
         builder.exclude(W3CTestCase.class, 
"(id=http://www.w3.org/2001/DOM-Test-Suite/level2/core/nodegetownerdocument01)");
         builder.exclude(W3CTestCase.class, 
"(id=http://www.w3.org/2001/DOM-Test-Suite/level2/core/nodegetownerdocument02)");
         builder.exclude(W3CTestCase.class, 
"(id=http://www.w3.org/2001/DOM-Test-Suite/level2/core/nodenormalize01)");
-        builder.exclude(W3CTestCase.class, 
"(id=http://www.w3.org/2001/DOM-Test-Suite/level2/core/nodesetprefix05)");
         builder.exclude(W3CTestCase.class, 
"(id=http://www.w3.org/2001/DOM-Test-Suite/level2/core/nodesetprefix06)");
         builder.exclude(W3CTestCase.class, 
"(id=http://www.w3.org/2001/DOM-Test-Suite/level2/core/nodesetprefix07)");
         builder.exclude(W3CTestCase.class, 
"(id=http://www.w3.org/2001/DOM-Test-Suite/level2/core/nodesetprefix09)");
diff --git 
a/mixins/dom-mixins/src/main/java/org/apache/axiom/dom/impl/mixin/DOMNSAwareNamedNodeMixin.java
 
b/mixins/dom-mixins/src/main/java/org/apache/axiom/dom/impl/mixin/DOMNSAwareNamedNodeMixin.java
index a855a7588..fa888f84c 100644
--- 
a/mixins/dom-mixins/src/main/java/org/apache/axiom/dom/impl/mixin/DOMNSAwareNamedNodeMixin.java
+++ 
b/mixins/dom-mixins/src/main/java/org/apache/axiom/dom/impl/mixin/DOMNSAwareNamedNodeMixin.java
@@ -21,6 +21,7 @@ package org.apache.axiom.dom.impl.mixin;
 import static org.apache.axiom.dom.DOMExceptionUtil.newDOMException;
 
 import org.apache.axiom.dom.DOMNSAwareNamedNode;
+import org.apache.axiom.dom.NSUtil;
 import org.apache.axiom.weaver.annotation.Mixin;
 import org.w3c.dom.DOMException;
 
@@ -51,6 +52,7 @@ public abstract class DOMNSAwareNamedNodeMixin implements 
DOMNSAwareNamedNode {
         if (coreGetNamespaceURI().length() == 0 && prefix.length() > 0) {
             throw newDOMException(DOMException.NAMESPACE_ERR);
         }
+        NSUtil.validatePrefix(prefix);
         coreSetPrefix(prefix);
     }