From 35aa668b87445f4616aaf0e6dad94d3d7fcc4264 Mon Sep 17 00:00:00 2001
From: ray <ray@6335cc39-0255-0410-8fd6-9bcaacd3b74c>
Date: Thu, 10 Mar 2005 22:29:50 +0000
Subject: [PATCH] More HTML restriction: block "<" from appearing in font, img,
 a href, and added <br>, <p>, and <hr> to the 'formatting' common regexes.

git-svn-id: https://samskivert.googlecode.com/svn/trunk@1612 6335cc39-0255-0410-8fd6-9bcaacd3b74c
---
 .../src/java/com/samskivert/util/StringUtil.java   | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/projects/samskivert/src/java/com/samskivert/util/StringUtil.java b/projects/samskivert/src/java/com/samskivert/util/StringUtil.java
index 7c16be0d..5f63e163 100644
--- a/projects/samskivert/src/java/com/samskivert/util/StringUtil.java
+++ b/projects/samskivert/src/java/com/samskivert/util/StringUtil.java
@@ -150,8 +150,9 @@ public class StringUtil
     /**
      * Restrict HTML except for the specified tags.
      * 
-     * @param allowFormatting enables &lt;i&gt;, &lt;b&gt;, &lt;u&gt;, and 
-     *                        &lt;font&gt;.
+     * @param allowFormatting enables &lt;i&gt;, &lt;b&gt;, &lt;u&gt;,
+     *                        &lt;font&gt;, &lt;br&gt;, &lt;p&gt;, and
+     *                        &lt;hr&gt;.
      * @param allowImages enabled &lt;img ...&gt;.
      * @param allowLinks enabled &lt;a href ...&gt;.
      */
@@ -165,13 +166,16 @@ public class StringUtil
             allow.add("<b>"); allow.add("</b>");
             allow.add("<i>"); allow.add("</i>");
             allow.add("<u>"); allow.add("</u>");
-            allow.add("<font [^>]+>"); allow.add("</font>");
+            allow.add("<font [^<>]+>"); allow.add("</font>");
+            allow.add("<br>"); allow.add("</br>");
+            allow.add("<p>"); allow.add("</p>");
+            allow.add("<hr>"); allow.add("</hr>");
         }
         if (allowImages) {
-            allow.add("<img [^>]+>"); allow.add("</img>");
+            allow.add("<img [^<>]+>"); allow.add("</img>");
         }
         if (allowLinks) {
-            allow.add("<a href=[^>]+>"); allow.add("</a>");
+            allow.add("<a href=[^<>]+>"); allow.add("</a>");
         }
 
         String[] regexes = new String[allow.size()];