From: martin f. krafft Date: Sun, 20 Apr 2008 21:23:38 GMT Subject: [PATCH] Escape project names before creating pathinfo URLs Message-ID: <1208726618-27477-1-git-send-email-madduck@madduck.net> In-Reply-To: <1208725436-25408-1-git-send-email-madduck@madduck.net> If a project name contains special URL characters like +, gitweb's links break in subtle ways. The solution is to pass the project name through esc_url() and using the return value. Signed-off-by: martin f. krafft --- gitweb/gitweb.perl | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl index a48bebb..241ae17 100755 --- a/gitweb/gitweb.perl +++ b/gitweb/gitweb.perl @@ -633,7 +633,7 @@ sub href(%) { my ($use_pathinfo) = gitweb_check_feature('pathinfo'); if ($use_pathinfo) { # use PATH_INFO for project name - $href .= "/$params{'project'}" if defined $params{'project'}; + $href .= "/".esc_url($params{'project'}) if defined $params{'project'}; delete $params{'project'}; # Summary just uses the project path URL @@ -2575,7 +2575,7 @@ EOF my $action = $my_uri; my ($use_pathinfo) = gitweb_check_feature('pathinfo'); if ($use_pathinfo) { - $action .= "/$project"; + $action .= "/".esc_url($project); } else { $cgi->param("p", $project); } -- 1.5.5.rc2