Skip to content
This repository has been archived by the owner on Feb 24, 2020. It is now read-only.

stage1: uid shift systemd files #3529

Merged
merged 1 commit into from
Jan 13, 2017
Merged

Conversation

lucab
Copy link
Member

@lucab lucab commented Jan 13, 2017

This commit adjusts some systemd-related files in stage1
which were not properly uid-shifted for in case of user-ns.

Fixes #3528

This commit adjusts some systemd-related files in stage1
which were not properly uid-shifted for in case of user-ns.
@lucab
Copy link
Member Author

lucab commented Jan 13, 2017

@jodh-intel can you please give this a try and see if there are any remaining spots?

@lucab lucab added this to the v1.23.0 milestone Jan 13, 2017
@jodh-intel
Copy link
Contributor

@lucab - thanks for the quick fix! Yes, I can confirm it works for me. I tested with umask values 027, 0000, 0111, 0222, 0444, 0777).

@lucab
Copy link
Member Author

lucab commented Jan 13, 2017

@jodh-intel thanks for confirming, hopefully this unblocks your other bug-fixing.

@squeed mind a review?

Copy link
Contributor

@s-urbaniak s-urbaniak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lucab lucab removed the request for review from squeed January 13, 2017 16:57
@lucab lucab merged commit 81602c6 into rkt:master Jan 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants