diff options
author | terry%netscape.com <> | 1999-01-20 02:17:21 +0100 |
---|---|---|
committer | terry%netscape.com <> | 1999-01-20 02:17:21 +0100 |
commit | 7613f7b0a655bf363ea80e511fe4f8364366a94f (patch) | |
tree | d8a57e69fb8d87d97162022725d9edda25aa14bd /buglist.cgi | |
parent | 09033034edd8cfc8aee44b37db86b5f823d19699 (diff) | |
download | bugzilla-7613f7b0a655bf363ea80e511fe4f8364366a94f.tar.gz bugzilla-7613f7b0a655bf363ea80e511fe4f8364366a94f.tar.xz |
Wasn't properly stripping out null queries on text searches.
Harmless, but was generating overly complicated SQL.
Diffstat (limited to 'buglist.cgi')
-rwxr-xr-x | buglist.cgi | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/buglist.cgi b/buglist.cgi index 4a59cbbe8..bc4fbc5cb 100755 --- a/buglist.cgi +++ b/buglist.cgi @@ -328,12 +328,13 @@ Click the <B>Back</B> button and try again."; foreach my $f ("short_desc", "long_desc", "bug_file_loc") { if (defined $::FORM{$f}) { - my $s = SqlQuote(trim($::FORM{$f})); + my $s = trim($::FORM{$f}); if ($s ne "") { + $s = SqlQuote($s); if ($::FORM{$f . "_type"} eq "regexp") { - $query .= "and $f regexp $s "; + $query .= "and $f regexp $s\n"; } else { - $query .= "and instr($f, $s) "; + $query .= "and instr($f, $s)\n"; } } } |