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: do not rely on DAC_OVERRIDE capability on Linux despite being root #69

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

Commits on Feb 18, 2019

  1. Fix: do not rely on DAC_OVERRIDE capability on Linux despite being root

    It turns out, for example, that SELinux will prevent this capability
    for confined processes running as root.  It then means that sbd cannot
    access, as a client, files used for joining two local communication sides
    within libqb-arranged IPC mechanism in case those files do not have
    permissions to explicitly allow file-based access with credentials
    of this client -- which is exactly what happens when the IPC servers
    are pacemaker daemons not run as root on their own.
    
    Solution is two-phased:
    1. have sbd add respective non-privileged group corresponding to the
       server side of the IPC -- this patch
    2. ensure this server side (pacemaker) does allow group-derived access
       (i.e., the access permissions for group are as relaxed as needed,
       umask notwithstanding) -- outside of the sbd's scope
    
    Signed-off-by: Jan Pokorný <[email protected]>
    jnpkrn committed Feb 18, 2019
    Configuration menu
    Copy the full SHA
    9695ba0 View commit details
    Browse the repository at this point in the history