Skip to content

Commit

Permalink
Revert "security-wrapper: Don't remove the old paths yet as that can …
Browse files Browse the repository at this point in the history
…create migration pain"

This reverts commit 4c751ce.

This does not fix the issue as /run is now mounted with nosuid.
  • Loading branch information
globin committed Mar 23, 2017
1 parent f9add54 commit 45f486f
Showing 1 changed file with 19 additions and 0 deletions.
19 changes: 19 additions & 0 deletions nixos/modules/security/wrappers/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -177,6 +177,25 @@ in
# programs to be wrapped.
WRAPPER_PATH=${config.system.path}/bin:${config.system.path}/sbin
# Remove the old /var/setuid-wrappers path from the system...
#
# TODO: this is only necessary for ugprades 16.09 => 17.x;
# this conditional removal block needs to be removed after
# the release.
if [ -d /var/setuid-wrappers ]; then
rm -rf /var/setuid-wrappers
fi
# Remove the old /run/setuid-wrappers-dir path from the
# system as well...
#
# TODO: this is only necessary for ugprades 16.09 => 17.x;
# this conditional removal block needs to be removed after
# the release.
if [ -d /run/setuid-wrapper-dirs ]; then
rm -rf /run/setuid-wrapper-dirs
fi
# We want to place the tmpdirs for the wrappers to the parent dir.
wrapperDir=$(mktemp --directory --tmpdir="${parentWrapperDir}" wrappers.XXXXXXXXXX)
chmod a+rx $wrapperDir
Expand Down

0 comments on commit 45f486f

Please sign in to comment.