gitweb: Fix fallback mode of to_utf8 subroutine

[git.git] / gitweb / gitweb.perl
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl

index d33510804ba34afa90c263cb20485d8fc3b0ce46..874023a33e9162b99bfa5b0e4038b102c129f6f1 100755 (executable)
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -1442,8 +1442,8 @@ sub validate_refname {
  sub to_utf8 {
         my $str = shift;
         return undef unless defined $str;
-       if (utf8::valid($str)) {
-               utf8::decode($str);
+
+       if (utf8::is_utf8($str) || utf8::decode($str)) {
                 return $str;
         } else {
                 return decode($fallback_encoding, $str, Encode::FB_DEFAULT);
@@ -1519,6 +1519,17 @@ sub esc_path {
         return $str;
  }
  
+# Sanitize for use in XHTML + application/xml+xhtm (valid XML 1.0)
+sub sanitize {
+       my $str = shift;
+
+       return undef unless defined $str;
+
+       $str = to_utf8($str);
+       $str =~ s|([[:cntrl:]])|($1 =~ /[\t\n\r]/ ? $1 : quot_cec($1))|eg;
+       return $str;
+}
+
  # Make control characters "printable", using character escape codes (CEC)
  sub quot_cec {
         my $cntrl = shift;
@@ -1684,6 +1695,7 @@ sub chop_and_escape_str {
         my ($str) = @_;
  
         my $chopped = chop_str(@_);
+       $str = to_utf8($str);
         if ($chopped eq $str) {
                 return esc_html($chopped);
         } else {
@@ -2877,7 +2889,7 @@ sub filter_forks_from_projects_list {
                 $path =~ s/\.git$//;      # forks of 'repo.git' are in 'repo/' directory
                 next if ($path =~ m!/$!); # skip non-bare repositories, e.g. 'repo/.git'
                 next unless ($path);      # skip '.git' repository: tests, git-instaweb
-               next unless (-d $path);   # containing directory exists
+               next unless (-d "$projectroot/$path"); # containing directory exists
                 $pr->{'forks'} = [];      # there can be 0 or more forks of project
  
                 # add to trie
@@ -6096,7 +6108,9 @@ sub git_blame_common {
                         -type=>"text/plain", -charset => "utf-8",
                         -status=> "200 OK");
                 local $| = 1; # output autoflush
-               print while <$fd>;
+               while (my $line = <$fd>) {
+                       print to_utf8($line);
+               }
                 close $fd
                         or print "ERROR $!\n";
  
@@ -6491,7 +6505,8 @@ sub git_blob {
                         $nr++;
                         $line = untabify($line);
                         printf qq!<div class="pre"><a id="l%i" href="%s#l%i" class="linenr">%4i</a> %s</div>\n!,
-                              $nr, esc_attr(href(-replay => 1)), $nr, $nr, $syntax ? to_utf8($line) : esc_html($line, -nbsp=>1);
+                              $nr, esc_attr(href(-replay => 1)), $nr, $nr,
+                              $syntax ? sanitize($line) : esc_html($line, -nbsp=>1);
                 }
         }
         close $fd
@@ -7687,11 +7702,12 @@ sub git_opml {
                 -charset => 'utf-8',
                 -content_disposition => 'inline; filename="opml.xml"');
  
+       my $title = esc_html($site_name);
         print <<XML;
  <?xml version="1.0" encoding="utf-8"?>
  <opml version="1.0">
  <head>
-  <title>$site_name OPML Export</title>
+  <title>$title OPML Export</title>
  </head>
  <body>
  <outline text="git RSS feeds">