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

common/hooks/post-install: add fix permissions hook #30139

Conversation

@paper42
Copy link
Contributor

@paper42 paper42 commented Apr 10, 2021

Some packages install files with wrong permissions, but sometimes we can detect and fix them.

/usr/share/man: 644

this rule matches a lot of files, mainly because it matches 444 permissions too

  • packages which install manpages with 755 permissions: nvimpager, sloccount
  • packages which install manpages with 444 permissions: lowdown, mdocml, dhcpcd, openresolv, all perl packages, lua5.3 (but not 5.1, 5.2 and 5.4)

/etc/apparmor.d: 600

I chose 600 because that's what aa-genprof creates.

  • packages which install apparmor profiles wrong permissions: apparmor (644), brillo (640), firejail (644), mako (640) (these permission measurements may be wrong in some cases)

The package lists are not complete.

Are there any other common directories which should be included in this hook? Is forcing 644 in /usr/share/man too strict? Should affected packages be revbumped?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant