Bug 122418 pt2 - obsoleting a patch from the create attachment screen gave

a taint error.
parent 90c95ff3
...@@ -314,7 +314,7 @@ sub validateObsolete ...@@ -314,7 +314,7 @@ sub validateObsolete
# Make sure the attachment id is valid and the user has permissions to view # Make sure the attachment id is valid and the user has permissions to view
# the bug to which it is attached. # the bug to which it is attached.
foreach my $attachid (@{$::MFORM{'obsolete'}}) { foreach my $attachid (@{$::MFORM{'obsolete'}}) {
$attachid =~ /^[1-9][0-9]*$/ detaint_natural($attachid)
|| DisplayError("The attachment number of one of the attachments || DisplayError("The attachment number of one of the attachments
you wanted to obsolete is invalid.") you wanted to obsolete is invalid.")
&& exit; && exit;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment