diff options
| author |   lpsolit%gmail.com <> | 2006-10-21 06:21:08 +0000 |
|---|---|---|
| committer | lpsolit%gmail.com <> | 2006-10-21 06:21:08 +0000 |
| commit | a39a4ef297b2435cbd4a4ef50d459b68c8bb6106 (patch) | |
| tree | 6331e2b53305d1dd8fb4a4ee7eab4bbbfdbbe42b /showdependencygraph.cgi | |
| parent | Bug 135812: Add a 'mailfrom' parameter to unify bugmail originating address -... (diff) | |
| download | bugzilla-a39a4ef297b2435cbd4a4ef50d459b68c8bb6106.tar.gz bugzilla-a39a4ef297b2435cbd4a4ef50d459b68c8bb6106.tar.bz2 bugzilla-a39a4ef297b2435cbd4a4ef50d459b68c8bb6106.zip | |
Bug 356328: $rankdir should be validated in showdependencygraph.cgi - Patch by Frédéric Buclin <LpSolit@gmail.com> r=bkor a=myk
Diffstat (limited to 'showdependencygraph.cgi')
| -rwxr-xr-x | showdependencygraph.cgi | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/showdependencygraph.cgi b/showdependencygraph.cgi index e483fd0f8..2cea9b2f2 100755 --- a/showdependencygraph.cgi +++ b/showdependencygraph.cgi @@ -96,7 +96,16 @@ sub AddLink { } } +# The list of valid directions. Some are not proposed in the dropdrown +# menu despite they are valid ones. +my @valid_rankdirs = ('LR', 'RL', 'TB', 'BT'); + my $rankdir = $cgi->param('rankdir') || "LR"; +# Make sure the submitted 'rankdir' value is valid. +if (lsearch(\@valid_rankdirs, $rankdir) < 0) { + $rankdir = 'LR'; +} + my $webdotdir = bz_locations()->{'webdotdir'}; if (!defined $cgi->param('id') && !defined $cgi->param('doall')) { |