aboutsummaryrefslogtreecommitdiff
path: root/gitweb/gitweb.perl
diff options
context:
space:
mode:
authorAndreas Brauchli <a.brauchli@elementarea.net>2016-07-29 16:49:37 +0200
committerJunio C Hamano <gitster@pobox.com>2016-08-01 12:55:40 -0700
commit77947bbe24e0306d1ce5605c962c4a25f5aca22f (patch)
tree008a9810b413fb41071fba1829c93d335f216cf2 /gitweb/gitweb.perl
parent0b65a8dbdb38962e700ee16776a3042beb489060 (diff)
downloadgit-77947bbe24e0306d1ce5605c962c4a25f5aca22f.tar.gz
git-77947bbe24e0306d1ce5605c962c4a25f5aca22f.tar.xz
gitweb: escape link body in format_ref_marker
Fix a case where an html link can be generated from unescaped input resulting in invalid strict xhtml or potentially injected code. An overview of a repo with a tag "1.0.0&0.0.1" would previously result in an unescaped ampersand in the link body. Signed-off-by: Andreas Brauchli <a.brauchli@elementarea.net> Acked-by: Jakub Narębski <jnareb@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'gitweb/gitweb.perl')
-rwxr-xr-xgitweb/gitweb.perl2
1 files changed, 1 insertions, 1 deletions
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 2fddf750f..33d701d85 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -2090,7 +2090,7 @@ sub format_ref_marker {
-href => href(
action=>$dest_action,
hash=>$dest
- )}, $name);
+ )}, esc_html($name));
$markers .= " <span class=\"".esc_attr($class)."\" title=\"".esc_attr($ref)."\">" .
$link . "</span>";