Commit 8ceb5a0b authored by Matt Tyson's avatar Matt Tyson Committed by Byron Jones

But 800196: Sanitise line-endings for textarea fields

parent 2a6f7d46
......@@ -152,6 +152,9 @@ sub VALIDATORS {
elsif ($field->type == FIELD_TYPE_BUG_ID) {
$validator = \&_check_bugid_field;
}
elsif ($field->type == FIELD_TYPE_TEXTAREA) {
$validator = \&_check_textarea_field;
}
else {
$validator = \&_check_default_field;
}
......@@ -2022,6 +2025,19 @@ sub _check_bugid_field {
return $checked_id;
}
sub _check_textarea_field {
my ($invocant, $text, $field) = @_;
$text = (defined $text) ? trim($text) : '';
# Web browsers submit newlines as \r\n.
# Sanitize all input to match the web standard.
# XMLRPC input could be either \n or \r\n
$text =~ s/\r?\n/\r\n/g;
return $text;
}
sub _check_relationship_loop {
# Generates a dependency tree for a given bug. Calls itself recursively
# to generate sub-trees for the bug's dependencies.
......
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