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

Regression in v252: missing ACL for volatile journal #402

Closed
debfx opened this issue May 26, 2024 · 0 comments
Closed

Regression in v252: missing ACL for volatile journal #402

debfx opened this issue May 26, 2024 · 0 comments

Comments

@debfx
Copy link

debfx commented May 26, 2024

systemd version the issue has been seen with

252.25-1~deb12u1

Used distribution

Debian 12

Linux kernel version used

6.1.0-21-amd64

CPU architectures issue was seen on

x86_64

Component

systemd-tmpfiles

Expected behaviour you didn't see

User who is (only) in the group adm is able to view the volatile journal stored in /run/log.

Unexpected behaviour you saw

journalctl -b returns -- No entries --

Steps to reproduce the problem

Set Storage=volatile in journald.conf, reboot and run journalctl -b as non-root user who is in the adm group.

Additional program output to the terminal or log subsystem illustrating the issue

On boot the following error is logged:


systemd-tmpfiles[259]: Failed to parse ACL "d:group:adm:r-x,group:adm:r-X": Invalid argument. Ignoring

I'm assuming this is because 19904f5 has been cherry-picked into 252 but support for X ACLs has only been added to 254: 26d98cd

The same problem likely exists in 253-stable as well but I haven't tested that.

@yuwata yuwata added the bug label May 26, 2024
bluca added a commit that referenced this issue May 26, 2024
The feature was introduced in v254, so 'X' is not available in this branch.

Fixes #402

This reverts commit 3cafb32.
bluca added a commit that referenced this issue May 26, 2024
The feature was introduced in v254, so 'X' is not available in this branch.

Fixes #402

This reverts commit 19904f5.
@bluca bluca closed this as completed May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants