[refpolicy] Debian has mailman lock files too
Daniel J Walsh
dwalsh at redhat.com
Wed Jul 1 11:54:06 CDT 2009
On 07/01/2009 11:21 AM, Manoj Srivastava wrote:
> diff --git a/policy/modules/services/mailman.fc b/policy/modules/services/mailman.fc
> index 839017f..3199d21 100644
> --- a/policy/modules/services/mailman.fc
> +++ b/policy/modules/services/mailman.fc
> @@ -31,3 +31,8 @@ ifdef(`distro_redhat', `
> /var/lock/mailman(/.*)? gen_context(system_u:object_r:mailman_lock_t,s0)
> /var/spool/mailman(/.*)? gen_context(system_u:object_r:mailman_data_t,s0)
> +ifdef(`distro_debian', `
> +/var/lock/mailman(/.*)? gen_context(system_u:object_r:mailman_lock_t,s0)
Why not remove the ifdef distro...*
We should not be adding ifdef distro unless the distros conflict on
labels. I don't imagine any distro is going to have /var/lock/mailman
be anything other them mailman_lock_t.
Several times I have had to move a label out of ifdef...debian because
fedora moved to the same labeling.
I think we should add as few ifdef(`disto into fc files as possible.
More information about the refpolicy