cherry-pick uri security fix

[git.ikiwiki.info.git] / IkiWiki / Plugin / map.pm

diff --git a/IkiWiki/Plugin/map.pm b/IkiWiki/Plugin/map.pm
index 93ff3194a53d57eaff8b11dd2e26b1faf1354371..581ae5011f8a002190f133f6ce999797bae43b1b 100644 (file)
--- a/IkiWiki/Plugin/map.pm
+++ b/IkiWiki/Plugin/map.pm
@@ -37,6 +37,9 @@ sub preprocess (@) { #{{{
                                my @b=split(/\//, $common_prefix);
                                $common_prefix="";
                                while (@a && @b && $a[0] eq $b[0]) {
+                                       if (length $common_prefix) {
+                                               $common_prefix.="/";
+                                       }
                                        $common_prefix.=shift(@a);
                                        shift @b;
                                }
@@ -45,7 +48,8 @@ sub preprocess (@) { #{{{
        }
        
        # Common prefix should not be a page in the map.
-       while (length $common_prefix && exists $mapitems{$common_prefix}) {
+       while (defined $common_prefix && length $common_prefix &&
+              exists $mapitems{$common_prefix}) {
                $common_prefix=IkiWiki::dirname($common_prefix);
        }
 
@@ -60,19 +64,23 @@ sub preprocess (@) { #{{{
        my $parent="";
        my $indent=0;
        my $openli=0;
+       my $dummy=0;
        my $map = "<div class='map'>\n<ul>\n";
        foreach my $item (sort keys %mapitems) {
-               $item=~s/^\Q$common_prefix\E\/// if length $common_prefix;
+               $item=~s/^\Q$common_prefix\E\///
+                       if defined $common_prefix && length $common_prefix;
                my $depth = ($item =~ tr/\//\//) + 1;
                my $baseitem=IkiWiki::dirname($item);
                while (length $parent && length $baseitem && $baseitem !~ /^\Q$parent\E(\/|$)/) {
                        $parent=IkiWiki::dirname($parent);
+                       last if !$dummy && length $parent && $baseitem =~ /^\Q$parent\E(\/|$)/;
                        $indent--;
                        $map .= "</li>\n";
                        if ($indent > 0) {
                                $map .= "</ul>\n";
                        }
                }
+               $dummy=0;
                while ($depth < $indent) {
                        $indent--;
                        $map .= "</li>\n";
@@ -89,9 +97,12 @@ sub preprocess (@) { #{{{
                                $map .= "<ul>\n";
                        }
                        if ($depth > $indent) {
+                               $dummy=1;
                                $p.="/".shift(@bits);
                                $map .= "<li>"
-                                       .htmllink($params{page}, $params{destpage}, $p, class => "mapparent")
+                                       .htmllink($params{page}, $params{destpage},
+                                                $p, class => "mapparent",
+                                                noimageinline => 1)
                                        ."\n";
                                $openli=1;
                        }
@@ -102,7 +113,8 @@ sub preprocess (@) { #{{{
                $map .= "</li>\n" if $openli;
                $map .= "<li>"
                        .htmllink($params{page}, $params{destpage}, 
-                               "/".$common_prefix."/".$item, class => "mapitem")
+                               "/".$common_prefix."/".$item,
+                               class => "mapitem", noimageinline => 1)
                        ."\n";
                $openli=1;
                $parent=$item;