Skip to content

enable usergroups and add pam_umask in common-session(-noninteractive)

Andreas Henriksson requested to merge ah/pam:ah/enable-usergroups into master

This merge-request contains a set of changes to enable the usergroups option of pam_umask by default as well as adding pam_umask.so as an optional session module.

There's also a commit that adds a new option nousergroups to the pam_umask module (and documents it) to allow users who don't want to use usergroups to disable it, but if you want to skip that one I don't mind at all as I'm personally just interested in a sane default.

These changes together will make sure that umask is reset when switching to a different user, eg. su - testuser will end up with umask 0002 no matter what your previous umask was. (And su - will end up with umask 0022 as root user is special cased, see usergroups in man pam_umask.) Previous behaviour is that the umask would be inherited. eg. if umask currently reported 0777 doing su - the new session would end up with umask 0777.

The bug reports in the debian bug tracking system that this fixes are mentioned in each commit, but also listed below for convenience. Please use gbp dch --auto to update debian/changelog after merging.

Merge request reports

Loading