summaryrefslogtreecommitdiffstats
path: root/report.cgi
diff options
context:
space:
mode:
authorFrédéric Buclin <LpSolit@gmail.com>2012-09-29 13:47:13 +0200
committerFrédéric Buclin <LpSolit@gmail.com>2012-09-29 13:47:13 +0200
commit268bc33484b25003db28f659d14328798e8830e4 (patch)
treeeb335ed91c379e911f895c099e044b2238ab6f7c /report.cgi
parent04313ae18ed5a139558aa7dad9f8bad23d13caa8 (diff)
downloadbugzilla-268bc33484b25003db28f659d14328798e8830e4.tar.gz
bugzilla-268bc33484b25003db28f659d14328798e8830e4.tar.xz
Bug 793893: Tabular reports crash when no format parameter is defined
r=glob a=LpSolit
Diffstat (limited to 'report.cgi')
-rwxr-xr-xreport.cgi9
1 files changed, 4 insertions, 5 deletions
diff --git a/report.cgi b/report.cgi
index 1ec3ac864..4230ade36 100755
--- a/report.cgi
+++ b/report.cgi
@@ -84,10 +84,12 @@ if (defined($height)) {
$height <= 2000 || ThrowUserError("chart_too_large");
}
+my $formatparam = $cgi->param('format') || '';
+
# These shenanigans are necessary to make sure that both vertical and
# horizontal 1D tables convert to the correct dimension when you ask to
# display them as some sort of chart.
-if (defined $cgi->param('format') && $cgi->param('format') eq "table") {
+if ($formatparam eq "table") {
if ($col_field && !$row_field) {
# 1D *tables* should be displayed vertically (with a row_field only)
$row_field = $col_field;
@@ -210,7 +212,7 @@ $vars->{'row_names'} = \@row_names;
$vars->{'tbl_names'} = \@tbl_names;
# Below a certain width, we don't see any bars, so there needs to be a minimum.
-if ($width && $cgi->param('format') eq "bar") {
+if ($width && $formatparam eq "bar") {
my $min_width = (scalar(@col_names) || 1) * 20;
if (!$cgi->param('cumulate')) {
@@ -226,8 +228,6 @@ $vars->{'height'} = $height if $height;
$vars->{'query'} = $query;
$vars->{'debug'} = $cgi->param('debug');
-my $formatparam = $cgi->param('format');
-
if ($action eq "wrap") {
# So which template are we using? If action is "wrap", we will be using
# no format (it gets passed through to be the format of the actual data),
@@ -236,7 +236,6 @@ if ($action eq "wrap") {
# data, or images generated by calling report.cgi again with action as
# "plot".
$formatparam =~ s/[^a-zA-Z\-]//g;
- trick_taint($formatparam);
$vars->{'format'} = $formatparam;
$formatparam = '';