login
Header Space

 
 

Re: gitweb search page link slightly wrong

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <git@...>
Date: Tuesday, May 1, 2007 - 6:13 am

This seems to fix it but I have no idea what it breaks. Command
injection should be stopped a few lines above that, and no other
parameter is ever quoted using quotemeta, so I'm not sure what the point
is, but I suppose it is actually necessary because the search text is
then wrapped into a regular expression or something?

--- git.orig/gitweb/gitweb.perl	2007-05-01 11:58:27.000000000 +0200
+++ git/gitweb/gitweb.perl	2007-05-01 12:11:56.000000000 +0200
@@ -368,7 +368,6 @@ if (defined $searchtext) {
 	if (length($searchtext) < 2) {
 		die_error(undef, "At least two characters are required for search parame=
ter");
 	}
-	$searchtext =3D quotemeta $searchtext;
 }
=20
 our $searchtype =3D $cgi->param('st');
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
gitweb search page link slightly wrong, Johannes Berg, (Fri Apr 27, 4:43 pm)
Re: gitweb search page link slightly wrong, Johannes Berg, (Tue May 1, 6:13 am)
Re: gitweb search page link slightly wrong, Johannes Berg, (Tue May 1, 6:22 am)
speck-geostationary