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

Fix apparmor config. #864

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

Fix apparmor config. #864

wants to merge 1 commit into from

Commits on Apr 18, 2022

  1. Fix apparmor config.

    Related bugs:
     * https://bugzilla.suse.com/show_bug.cgi?id=1111906
     * https://bugs.debian.org/988068
     * jonls#672
     * jonls#860
    
    src/config-ini.c still looks for $HOME/.config/redshift.conf which
    fails to open because the apparmor profile doesn’t allow it.
    
    src/config-ini.c also does this:
       struct passwd *pwd = getpwuid(getuid());
       char *home = pwd->pw_dir;
    
    Since getpwuid doesn’t have access to /etc/passwd* it returns NULL,
    causing a segmentation fault in the line thereafter.
    This code is not triggered if a config is specified (e.g.
    -c redshift.conf) or $HOME/.config/redshift.conf exists.
    rumpeltux committed Apr 18, 2022
    Configuration menu
    Copy the full SHA
    f4d5d4d View commit details
    Browse the repository at this point in the history