Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: Don't assume uid == primary gid #1390

Closed
wants to merge 1 commit into from
Closed

Conversation

smcv
Copy link
Contributor

@smcv smcv commented Jan 2, 2018

Nothing guarantees that each user has a group containing only
themselves. Even if they do, nothing guarantees that its group ID
equals the user ID, particularly if another user earlier in the same
range was created without a corresponding group or vice versa.

Signed-off-by: Simon McVittie [email protected]


Applied in Debian since 2017.14.

Nothing guarantees that each user has a group containing only
themselves. Even if they do, nothing guarantees that its group ID
equals the user ID, particularly if another user earlier in the same
range was created without a corresponding group or vice versa.

Signed-off-by: Simon McVittie <[email protected]>
@cgwalters
Copy link
Member

Hm, from test-concurrency.py in Travis:

error: opendir(staging-dc9bf54c-5955-4738-9a33-ecc0017fb577-jv0bfY): No such file or directory

May be a new race 🏃‍♂️. Will look at that later. Thanks for the patch!

@rh-atomic-bot r+ e3ed68b

@rh-atomic-bot
Copy link

⚡ Test exempted: merge already tested.

@smcv smcv deleted the uid-ne-gid branch January 2, 2018 14:52
raspbian-autopush pushed a commit to raspbian-packages/ostree that referenced this pull request Jan 11, 2018
Nothing guarantees that each user has a group containing only
themselves. Even if they do, nothing guarantees that its group ID
equals the user ID, particularly if another user earlier in the same
range was created without a corresponding group or vice versa.

Signed-off-by: Simon McVittie <[email protected]>
Forwarded: ostreedev/ostree#1390
Applied-upstream: 2018.1, commit:ostreedev/ostree@f63e62f

Gbp-Pq: Topic 2018.1
Gbp-Pq: Name tests-Don-t-assume-uid-primary-gid.patch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants