aboutsummaryrefslogtreecommitdiffstats
path: root/post_bug.cgi
diff options
context:
space:
mode:
authormozilla%colinogilvie.co.uk <>2006-01-09 03:56:03 +0000
committermozilla%colinogilvie.co.uk <>2006-01-09 03:56:03 +0000
commit8508ede5763d9d98d8c4fc755847752d192c2126 (patch)
tree52943273ee4f76b7121d9568ce528e585d3c44a2 /post_bug.cgi
parent94232797b7ea157c75ee92a9830993fbff1caad1 (diff)
downloadbugs-8508ede5763d9d98d8c4fc755847752d192c2126.tar
bugs-8508ede5763d9d98d8c4fc755847752d192c2126.tar.gz
bugs-8508ede5763d9d98d8c4fc755847752d192c2126.tar.bz2
bugs-8508ede5763d9d98d8c4fc755847752d192c2126.tar.xz
bugs-8508ede5763d9d98d8c4fc755847752d192c2126.zip
Bug 101380: Newlines, nulls, leading/trailing spaces are getting into summaries
Patch by Paul <pdemarco@zoominternet.net> and Colin Ogilvie <colin.ogilvie@gmail.com>; r/a=justdave
Diffstat (limited to 'post_bug.cgi')
-rwxr-xr-xpost_bug.cgi5
1 files changed, 4 insertions, 1 deletions
diff --git a/post_bug.cgi b/post_bug.cgi
index 3d2d6ab48..4d8c6a2c9 100755
--- a/post_bug.cgi
+++ b/post_bug.cgi
@@ -108,8 +108,11 @@ my $component_id = get_component_id($product_id,
scalar($cgi->param('component')));
$component_id || ThrowUserError("require_component");
+# Set the parameter to itself, but cleaned up
+$cgi->param('short_desc', clean_text($cgi->param('short_desc')));
+
if (!defined $cgi->param('short_desc')
- || trim($cgi->param('short_desc')) eq "") {
+ || $cgi->param('short_desc') eq "") {
ThrowUserError("require_summary");
}