From c96d672810ae524aedf9ffff08dcca773cd9a876 Mon Sep 17 00:00:00 2001 From: joey Date: Wed, 25 Jul 2007 03:36:53 +0000 Subject: massive patchqueue reorg patches can now be anywhere and tagged patch to show up on the patch list. Moved all the patchqueue stuff to todo items; some of it was merged into existing todo items. --- doc/todo/Wrapper_config_with_multiline_regexp.mdwn | 36 ++++++++++++++++++++++ 1 file changed, 36 insertions(+) create mode 100644 doc/todo/Wrapper_config_with_multiline_regexp.mdwn (limited to 'doc/todo/Wrapper_config_with_multiline_regexp.mdwn') diff --git a/doc/todo/Wrapper_config_with_multiline_regexp.mdwn b/doc/todo/Wrapper_config_with_multiline_regexp.mdwn new file mode 100644 index 000000000..c0311bc92 --- /dev/null +++ b/doc/todo/Wrapper_config_with_multiline_regexp.mdwn @@ -0,0 +1,36 @@ +Turning the wikilink regexp into an extended regexp on the svn trunk seems +to have broken the setuid wrapper on my system, because of two reasons: +First, the wrapper generator should turn each newline in $configstring into +`\n` in the C code rather than `\` followed by a newline in the C code. +Second, the untainting of $configstring should allow newlines. + +> Both of these problems were already dealt with in commit r3714, on June +> 3rd. Confused why you're posting patches for them now. [[done]] --[[Joey]] + + Modified: wiki-meta/perl/IkiWiki.pm + ============================================================================== + --- wiki-meta/perl/IkiWiki.pm (original) + +++ wiki-meta/perl/IkiWiki.pm Mon Jun 11 10:52:07 2007 + @@ -205,7 +205,7 @@ + + sub possibly_foolish_untaint ($) { #{{{ + my $tainted=shift; + - my ($untainted)=$tainted=~/(.*)/; + + my ($untainted)=$tainted=~/(.*)/s; + return $untainted; + } #}}} + + + Modified: wiki-meta/perl/IkiWiki/Wrapper.pm + ============================================================================== + --- wiki-meta/perl/IkiWiki/Wrapper.pm (original) + +++ wiki-meta/perl/IkiWiki/Wrapper.pm Mon Jun 11 10:52:07 2007 + @@ -62,7 +62,7 @@ + } + $configstring=~s/\\/\\\\/g; + $configstring=~s/"/\\"/g; + - $configstring=~s/\n/\\\n/g; + + $configstring=~s/\n/\\n/g; + + #translators: The first parameter is a filename, and the second is + #translators: a (probably not translated) error message. -- cgit v1.2.3