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

New profile: ncmpcpp #6587

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

Conversation

amano-kenji
Copy link
Contributor

@amano-kenji amano-kenji commented Dec 27, 2024

Comment on lines +27 to +30
mkdir ${HOME}/.config/ncmpcpp
mkdir ${HOME}/.lyrics
whitelist ${HOME}/.config/ncmpcpp
whitelist ${HOME}/.lyrics
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add these paths to disable-programs.inc and add corresponding noblacklist
entries in this profile.

This should be done whenever adding program-specific paths.

Also, if any of the configuration files allow arbitrary code execution, add
read-only $path to disable-common.inc (and read-write $path to this profile
if the configuration file can be edited by the program itself).

This also applies to the other PRs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a lot of suggested changes. I will try to apply all changes at once tomorrow.

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