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. Cc: Jakub Narebski <jnareb@xxxxxxxxx> Signed-off-by: Petr Baudis <pasky@xxxxxxx> --- gitweb/gitweb.perl | 3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl index 677f4fb..1ac4523 100755 --- a/gitweb/gitweb.perl +++ b/gitweb/gitweb.perl @@ -476,9 +476,6 @@ if (defined $searchtype) { 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"); } - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html