urosm
/
ikiwiki
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

* ikiwiki-mass-rebuild: Patch from HenrikBrixAndersen to fix order 

* ikiwiki-mass-rebuild: Don't clear PATH from the environment.
master
Joey Hess 2007-10-25 23:12:23 -04:00
parent 11968615df
commit 1e1e1f5046
3 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
debian/changelog vendored
View File

@ -16,10 +16,11 @@ ikiwiki (2.11) UNRELEASED; urgency=low
* Add handling of feeds for nested inlines, as well as support for a
single page containing two different feeds.
* Also fixed some places in inline that failed to use destpage correctly.
* ikiwiki-mass-rebuild: Patch from HenrikBrixAndersen to fix order
* ikiwiki-mass-rebuild: Patch from HenrikBrixAndersen to fix order
of permissions dropping code to work on FreeBSD.
* ikiwiki-mass-rebuild: Don't clear PATH from the environment.
-- Joey Hess <joeyh@debian.org> Thu, 25 Oct 2007 07:52:43 -0400
-- Joey Hess <joeyh@debian.org> Thu, 25 Oct 2007 23:11:13 -0400
ikiwiki (2.10) unstable; urgency=low

2
doc/bugs/ikiwiki-mass-rebuild_fails_to_drop_privileges_and_execute_ikiwiki.mdwn
View File

@ -6,6 +6,8 @@ The solution is to set the effective UID after setting the real UID, and to set
> > It doesn't - it needs to be set. The line with `%ENV=();` clears the environment, thus no `$PATH` is set. --[[HenrikBrixAndersen]]
> > > I guess it shouldn't clear it then. Both [[done]] --[[Joey]]
Proposed patch:
--- ikiwiki-mass-rebuild.orig 2007-08-15 22:21:59.000000000 +0200

6
ikiwiki-mass-rebuild
View File

@ -27,8 +27,10 @@ sub processline {
if ($< != $uuid || $> != $uuid || $( != $ugid || $) ne "$ugid $ugid") {
die "failed to drop permissions to $user";
}
%ENV=();
$ENV{HOME}=(getpwnam($user))[7];
%ENV=(
PATH => $ENV{PATH},
HOME => (getpwnam($user))[7],
);
exec("ikiwiki", "-setup", $setup, @ARGV);
die "failed to run ikiwiki: $!";
}