commit bc12accbb9ee6caf2d6159dc484f74d33ab929bb
author Mark Thomas <markt@apache.org> Wed May 24 14:40:48 2023 +0100
committer Mark Thomas <markt@apache.org> Wed May 24 14:40:48 2023 +0100
tree ad69032f0a653549faa4a9e4315fd4c09f5fdf6d
parent 9a6bc65e50df5b8d138b5373e48575d043981c24

Reduce code duplication

java/org/apache/catalina/manager/JspHelper.java

diff --git a/java/org/apache/catalina/manager/JspHelper.java b/java/org/apache/catalina/manager/JspHelper.java
index cb6db56..bc0945c 100644
--- a/java/org/apache/catalina/manager/JspHelper.java
+++ b/java/org/apache/catalina/manager/JspHelper.java
@@ -24,6 +24,7 @@
 
 import org.apache.catalina.Session;
 import org.apache.catalina.manager.util.SessionUtils;
+import org.apache.tomcat.util.security.Escape;
 
 
 /**
@@ -191,57 +192,26 @@
     }
 
     /**
-     * Performs the following substring replacements
-     * (to facilitate output to XML/HTML pages):
+     * Performs the following substring replacements (to facilitate output to XML/HTML pages):
+     * <ul>
+     *   <li>&amp; -&gt; &amp;amp;</li>
+     *   <li>&lt; -&gt; &amp;lt;</li>
+     *   <li>&gt; -&gt; &amp;gt;</li>
+     *   <li>" -&gt; &amp;#034;</li>
+     *   <li>' -&gt; &amp;#039;</li>
+     * </ul>
      *
-     *    &amp; -&gt; &amp;amp;
-     *    &lt; -&gt; &amp;lt;
-     *    &gt; -&gt; &amp;gt;
-     *    " -&gt; &amp;#034;
-     *    ' -&gt; &amp;#039;
-     *
-     * See also OutSupport.writeEscapedXml().
      * @param buffer The XML to escape
+     *
      * @return the escaped XML
      */
-    @SuppressWarnings("null") // escapedBuffer cannot be null
     public static String escapeXml(String buffer) {
+
         if (buffer == null) {
             return "";
         }
-        int start = 0;
-        int length = buffer.length();
-        char[] arrayBuffer = buffer.toCharArray();
-        StringBuilder escapedBuffer = null;
 
-        for (int i = 0; i < length; i++) {
-            char c = arrayBuffer[i];
-            if (c <= HIGHEST_SPECIAL) {
-                char[] escaped = specialCharactersRepresentation[c];
-                if (escaped != null) {
-                    // create StringBuilder to hold escaped xml string
-                    if (start == 0) {
-                        escapedBuffer = new StringBuilder(length + 5);
-                    }
-                    // add unescaped portion
-                    if (start < i) {
-                        escapedBuffer.append(arrayBuffer,start,i-start);
-                    }
-                    start = i + 1;
-                    // add escaped xml
-                    escapedBuffer.append(escaped);
-                }
-            }
-        }
-        // no xml escaping was necessary
-        if (start == 0) {
-            return buffer;
-        }
-        // add rest of unescaped portion
-        if (start < length) {
-            escapedBuffer.append(arrayBuffer,start,length-start);
-        }
-        return escapedBuffer.toString();
+        return Escape.xml(buffer);
     }
 
     public static String formatNumber(long number) {