From 09dd14ac8ca604c4bc2bd3a3b3507e4f37bebec9 Mon Sep 17 00:00:00 2001 From: Byron Jones Date: Wed, 28 Dec 2011 16:57:33 -0500 Subject: [PATCH] Bug 697699 - (CVE-2011-3657) [SECURITY] XSS when viewing new charts or tabular and graphical reports in debug mode r=gerv, a=LpSolit --- Bugzilla/Chart.pm | 2 +- report.cgi | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Bugzilla/Chart.pm b/Bugzilla/Chart.pm index 760db135df..dfbf32a51d 100644 --- a/Bugzilla/Chart.pm +++ b/Bugzilla/Chart.pm @@ -438,7 +438,7 @@ sub dump { require Data::Dumper; print "
Bugzilla::Chart object:\n";
-    print Data::Dumper::Dumper($self);
+    print html_quote(Data::Dumper::Dumper($self));
     print "
"; } diff --git a/report.cgi b/report.cgi index 9695d54da2..a293bb6bc4 100755 --- a/report.cgi +++ b/report.cgi @@ -293,9 +293,9 @@ print $cgi->header(-type => $format->{'ctype'}, if ($cgi->param('debug')) { require Data::Dumper; print "
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
"; + print html_quote(Data::Dumper::Dumper(@image_data)) . "\n\n"; } # All formats point to the same section of the documentation. -- 2.47.2