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

[DRAFT] Fix regression: specifying IPC privileges using UID #630

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Cropi
Copy link
Member

@Cropi Cropi commented Jun 17, 2024

Recently, I came across an observation where it's not possible to specify the IPCAccessControlFiles using an UID instead of username/groupname. For example:

usbguard add-user 1000 --device list

Expected results: user with ID of 1000 can list devices
Actual results: ERROR: IPC access control: invalid name format: 1002

That is because the isValidName() function does not allow having 'digit-only' names. I haven't experimented with the patch, will leave it here for discussion.

This was working in the past. Not sure if anyone is using it like this, so another possibility could be to leave it as it is and modify the documentation.

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.

1 participant