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.

Sign up for GitHub

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

Jump to bottom

Make sure all backends which need root have correct permissions #7

Closed
tillkamppeter opened this issue Mar 5, 2021 · 2 comments
Closed

Make sure all backends which need root have correct permissions #7

tillkamppeter opened this issue Mar 5, 2021 · 2 comments

Comments

@tillkamppeter
Copy link
Member

Some CUPS backends (like "usb") need to run as root, they need to have execute permissions for non-root removed.
@tillkamppeter
Copy link
Member Author

The only backend with wrong permissons was the "usb" backend as CUPS upstream assumes that root is not needed here because UDEV sets "lp" group ownership to the raw USB devices. This does not work in our CUPS Snap. Therefore the "usb" backend has to run as root. CUPS controls this by the executable permissions, if only the owner (which is root) can run it, it runs as root.

This I have preliminary implemented in commit 0dbc1fb.

The best solution is to do it in CUPS upstream, but I have to find out how to best do it there. Therefore I will not close this issue for now.

@tillkamppeter
Copy link
Member Author

Solved in CUPS upstream, OpenPrinting/cups#121, workaround in the CUPS Snap removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tillkamppeter