From cdafdd00be2d834fb6e2b36b9275a07fad1036d5 Mon Sep 17 00:00:00 2001
From: Jamie Cameron <jcameron@webmin.com>
Date: Wed, 29 May 2013 17:18:34 -0700
Subject: [PATCH] Fix HTML escaping and trunctation
 http://virtualmin.com/node/27122

---
 bind8/edit_recs.cgi | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/bind8/edit_recs.cgi b/bind8/edit_recs.cgi
index dc3767f71..2cc2b0cb1 100755
--- a/bind8/edit_recs.cgi
+++ b/bind8/edit_recs.cgi
@@ -192,27 +192,27 @@ for($i=0; $i<@_; $i++) {
 	for($j=0; $j<@hmap; $j++) {
 		local $v;
 		if ($in{'type'} eq "RP" && $j == 0) {
-			$v .= &convert_illegal(
-				&dotted_to_email($r->{'values'}->[$j]));
+			$v .= &dotted_to_email($r->{'values'}->[$j]);
 			}
 		elsif ($in{'type'} eq "WKS" && $j == @hmap-1) {
 			for($k=$j; $r->{'values'}->[$k]; $k++) {
-				$v .= &convert_illegal($r->{'values'}->[$k]);
+				$v .= $r->{'values'}->[$k];
 				$v .= ' ';
 				}
 			}
 		elsif ($in{'type'} eq "LOC") {
-			$v = &convert_illegal(join(" ", @{$r->{'values'}}));
+			$v = join(" ", @{$r->{'values'}});
 			}
 		elsif ($in{'type'} eq "KEY" && $j == 3) {
 			$v = substr($r->{'values'}->[$j], 0, 20)."...";
 			}
 		else {
-			$v = &convert_illegal($r->{'values'}->[$j]);
+			$v = $r->{'values'}->[$j];
 			}
 		if (length($v) > 80) {
 			$v = substr($v, 0, 80)." ...";
 			}
+		$v = &html_escape($v);
 		push(@cols, $v);
 		}
 	if ($in{'type'} eq "ALL" || $is_extra{$in{'type'}}) {