Skip to content

Commit

Permalink
gitweb: Lift any characters restriction on searched strings
Browse files Browse the repository at this point in the history
Everything is already fully quoted along the way so I believe this to be
unnecessary at this point. It would pose trouble for regexp searches.

Signed-off-by: Petr Baudis <pasky@suse.cz>
Acked-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Petr Baudis authored and gitster committed Aug 27, 2007
1 parent 17437d4 commit 7d47962
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions gitweb/gitweb.perl
Original file line number Diff line number Diff line change
Expand Up @@ -471,9 +471,6 @@ sub filter_snapshot_fmts {
our $searchtext = $cgi->param('s');
our $search_regexp;
if (defined $searchtext) {
if ($searchtype ne 'grep' and $searchtype ne 'pickaxe' and $searchtext =~ m/[^a-zA-Z0-9_\.\/\-\+\:\@ ]/) {
die_error(undef, "Invalid search parameter");
}
if (length($searchtext) < 2) {
die_error(undef, "At least two characters are required for search parameter");
}
Expand Down

0 comments on commit 7d47962

Please sign in to comment.