From f19198cbce45f22bbfa8ef3816a8e99304dd9aae Mon Sep 17 00:00:00 2001
From: elizabeth <elizabeth@6335cc39-0255-0410-8fd6-9bcaacd3b74c>
Date: Mon, 8 Aug 2005 23:12:59 +0000
Subject: [PATCH] Tightened up restrictHTML regexes to prevent <\!-- or
 unclosed " html defacement

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

diff --git a/projects/samskivert/src/java/com/samskivert/util/StringUtil.java b/projects/samskivert/src/java/com/samskivert/util/StringUtil.java
index 394e4865..9f75d04d 100644
--- a/projects/samskivert/src/java/com/samskivert/util/StringUtil.java
+++ b/projects/samskivert/src/java/com/samskivert/util/StringUtil.java
@@ -166,16 +166,20 @@ 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>");
+            // Until I find a way to disallow "---", no - can be in a url
+            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()];