diff options
| author | mozilla%colinogilvie.co.uk <> | 2006-01-09 04:56:03 +0100 |
|---|---|---|
| committer | mozilla%colinogilvie.co.uk <> | 2006-01-09 04:56:03 +0100 |
| commit | 8508ede5763d9d98d8c4fc755847752d192c2126 (patch) | |
| tree | 52943273ee4f76b7121d9568ce528e585d3c44a2 /post_bug.cgi | |
| parent | 94232797b7ea157c75ee92a9830993fbff1caad1 (diff) | |
| download | bugzilla-8508ede5763d9d98d8c4fc755847752d192c2126.tar.gz bugzilla-8508ede5763d9d98d8c4fc755847752d192c2126.tar.xz | |
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-x | post_bug.cgi | 5 |
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"); } |