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

add passwd to private-etc #4193

Merged
merged 1 commit into from
Apr 14, 2021
Merged

add passwd to private-etc #4193

merged 1 commit into from
Apr 14, 2021

Conversation

glitsj16
Copy link
Collaborator

Now we have the new quodlibet.profile I've tested its functionality on a setup where users only have exfalso, but not quodlibet. Both profiles are looking good, but I noticed the below error when firejailing exfalso:

...
/usr/lib/python3.9/site-packages/quodlibet/qltk/filesel.py:130: Warning: getpwuid_r(): failed due to unknown user id (1001)
...

Adding passwd to private-etc fixes this for uid's > 1000.

@glitsj16 glitsj16 merged commit 8cb6c1e into netblue30:master Apr 14, 2021
@glitsj16 glitsj16 deleted the quodlibet branch April 14, 2021 20:29
@rusty-snake
Copy link
Collaborator

Adding passwd to private-etc fixes this for uid's > 1000.

Has this anything to do with the UID? All getpw* can not work without /etc/passwd because they are the API for it.

@glitsj16
Copy link
Collaborator Author

Has this anything to do with the UID? All getpw* can not work without /etc/passwd because they are the API for it.

I stand corrected. Happy to realize that by adding passwd to private-etc the PR at least didn't break anything :-D

@matu3ba matu3ba mentioned this pull request Oct 7, 2021
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.

2 participants