diff options
| author | Frédéric Buclin <LpSolit@gmail.com> | 2016-03-09 23:06:28 +0100 |
|---|---|---|
| committer | Frédéric Buclin <LpSolit@gmail.com> | 2016-03-09 23:06:28 +0100 |
| commit | eb323eb6db292aa9fb705ceaff9c94b3a7a74c4d (patch) | |
| tree | cad3064d2af051f59066b86ace9326bba218a0b2 /process_bug.cgi | |
| parent | 458aca660809144eb9b2c3d84cff103d00dac7c2 (diff) | |
| download | bugs-eb323eb6db292aa9fb705ceaff9c94b3a7a74c4d.tar bugs-eb323eb6db292aa9fb705ceaff9c94b3a7a74c4d.tar.gz bugs-eb323eb6db292aa9fb705ceaff9c94b3a7a74c4d.tar.bz2 bugs-eb323eb6db292aa9fb705ceaff9c94b3a7a74c4d.tar.xz bugs-eb323eb6db292aa9fb705ceaff9c94b3a7a74c4d.zip | |
Bug 1234977: Replace \d+ by [0-9]+ in critical validation places
r=dylan a=dkl
Diffstat (limited to 'process_bug.cgi')
| -rwxr-xr-x | process_bug.cgi | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/process_bug.cgi b/process_bug.cgi index 216dfbf1b..0b0ecd64e 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -305,9 +305,10 @@ if (defined $cgi->param('id')) { my %is_private; foreach my $field (grep(/^defined_isprivate/, $cgi->param())) { - $field =~ /(\d+)$/; - my $comment_id = $1; - $is_private{$comment_id} = $cgi->param("isprivate_$comment_id"); + if ($field =~ /(\d+)$/) { + my $comment_id = $1; + $is_private{$comment_id} = $cgi->param("isprivate_$comment_id"); + } } $set_all_fields{comment_is_private} = \%is_private; |