diff options
| author | Joey Hess <joey@kitenet.net> | 2010-06-18 12:48:05 -0400 |
|---|---|---|
| committer | Joey Hess <joey@kitenet.net> | 2010-06-18 12:48:05 -0400 |
| commit | 2797a659db82dd1a0021dacd005fa80212c41579 (patch) | |
| tree | 785cd9ce22c0206aac701cbfee15391dad796569 /IkiWiki | |
| parent | cb4b99929757f970d5ae697f0d09514ad624ed46 (diff) | |
| download | ikiwiki-2797a659db82dd1a0021dacd005fa80212c41579.tar ikiwiki-2797a659db82dd1a0021dacd005fa80212c41579.tar.gz | |
mercurial: Fix buggy getctime code.
The file passed to rcs_getctime is already absolute, and it was
trying to stick the srcdir on the front.
Also, eliminated potentially unsafe shelling.
Diffstat (limited to 'IkiWiki')
| -rw-r--r-- | IkiWiki/Plugin/mercurial.pm | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/IkiWiki/Plugin/mercurial.pm b/IkiWiki/Plugin/mercurial.pm index 34e009c7a..a80bb2da5 100644 --- a/IkiWiki/Plugin/mercurial.pm +++ b/IkiWiki/Plugin/mercurial.pm @@ -236,15 +236,13 @@ sub rcs_diff ($) { sub rcs_getctime ($) { my ($file) = @_; - # XXX filename passes through the shell here, should try to avoid - # that just in case my @cmdline = ("hg", "-R", $config{srcdir}, "log", "-v", - "--style", "default", "$config{srcdir}/$file"); - open (my $out, "@cmdline |"); + "--style", "default", $file); + open (my $out, "-|", @cmdline); - my @log = mercurial_log($out); + my @log = (mercurial_log($out)); - if (length @log < 1) { + if (@log < 1) { return 0; } |