Bug 697699 - (CVE-2011-3657) [SECURITY] XSS when viewing new charts or tabular and...

r=gerv, a=LpSolit

diff --git a/Bugzilla/Chart.pm b/Bugzilla/Chart.pm
index 760db135dfb67e51409db891955450f3ffe83d09..dfbf32a51d1cff01e590536e291a1d05f3e9174d 100644 (file)
--- a/Bugzilla/Chart.pm
+++ b/Bugzilla/Chart.pm
@@ -438,7 +438,7 @@ sub dump {
     
     require Data::Dumper;
     print "<pre>Bugzilla::Chart object:\n";
-    print Data::Dumper::Dumper($self);
+    print html_quote(Data::Dumper::Dumper($self));
     print "</pre>";
 }
 

diff --git a/report.cgi b/report.cgi
index 9695d54da2b038c4301cc561ac5f9edfdea55620..a293bb6bc43a26c0c63455852bf1602c838eb68f 100755 (executable)
--- a/report.cgi
+++ b/report.cgi
@@ -293,9 +293,9 @@ print $cgi->header(-type => $format->{'ctype'},
 if ($cgi->param('debug')) {
     require Data::Dumper;
     print "<pre>data hash:\n";
-    print Data::Dumper::Dumper(%data) . "\n\n";
+    print html_quote(Data::Dumper::Dumper(%data)) . "\n\n";
     print "data array:\n";
-    print Data::Dumper::Dumper(@image_data) . "\n\n</pre>";
+    print html_quote(Data::Dumper::Dumper(@image_data)) . "\n\n</pre>";
 }
 
 # All formats point to the same section of the documentation.