All of lore.kernel.org
 help / color / mirror / Atom feed
From: dac.override@gmail.com (Dominick Grift)
To: refpolicy@oss.tresys.com
Subject: [refpolicy] [PATCH v2 1/6] Courier TCPd startup creates imapd.pid.lock and imapd.lock
Date: Thu, 1 Jan 2015 18:41:21 +0100	[thread overview]
Message-ID: <20150101174121.GA4840@bigboy.network2> (raw)
In-Reply-To: <1420042198-4676-2-git-send-email-sven.vermeulen@siphos.be>

On Wed, Dec 31, 2014 at 05:09:53PM +0100, Sven Vermeulen wrote:
> Startup of courier-imapd creates /var/run/imapd.pid.lock and imapd.lock
> 
> Signed-off-by: Sven Vermeulen <sven.vermeulen@siphos.be>
> ---
>  courier.fc | 2 ++
>  courier.te | 2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/courier.fc b/courier.fc
> index 2f017a0..abdc37e 100644
> --- a/courier.fc
> +++ b/courier.fc
> @@ -27,6 +27,8 @@
>  /var/lib/courier-imap(/.*)?	gen_context(system_u:object_r:courier_var_lib_t,s0)
>  
>  /var/run/courier(/.*)?	gen_context(system_u:object_r:courier_var_run_t,s0)
> +/var/run/imapd\.pid	--	gen_context(system_u:object_r:courier_var_run_t,s0)

This conflicts with uwimap pid file context specification

Not sure how to deal with this, but i would prefer:

Did you build with: --with-piddir=dir - use dir/imapd.pid to store couriertcpd's process ID.

Probably better to set --with-piddir=/var/run/courier

That will make this patch redundant

> +/var/run/imapd\.pid\.lock	--	gen_context(system_u:object_r:courier_var_run_t,s0)
>  
>  /var/spool/authdaemon(/.*)?	gen_context(system_u:object_r:courier_spool_t,s0)
>  /var/spool/courier(/.*)?	gen_context(system_u:object_r:courier_spool_t,s0)
> diff --git a/courier.te b/courier.te
> index ae3bc70..112a60b 100644
> --- a/courier.te
> +++ b/courier.te
> @@ -172,6 +172,8 @@ corenet_tcp_sendrecv_pop_port(courier_tcpd_t)
>  dev_read_rand(courier_tcpd_t)
>  dev_read_urand(courier_tcpd_t)
>  
> +files_pid_filetrans(courier_tcpd_t, courier_var_run_t, file)
> +
>  miscfiles_read_localization(courier_tcpd_t)
>  
>  ########################################
> -- 
> 2.0.5
> 
> _______________________________________________
> refpolicy mailing list
> refpolicy at oss.tresys.com
> http://oss.tresys.com/mailman/listinfo/refpolicy

-- 
Dominick Grift
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 648 bytes
Desc: not available
Url : http://oss.tresys.com/pipermail/refpolicy/attachments/20150101/26b55a01/attachment.bin 

  reply	other threads:[~2015-01-01 17:41 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-31 16:09 [refpolicy] [PATCH v2 0/6] Courier IMAPd related policy changes Sven Vermeulen
2014-12-31 16:09 ` [refpolicy] [PATCH v2 1/6] Courier TCPd startup creates imapd.pid.lock and imapd.lock Sven Vermeulen
2015-01-01 17:41   ` Dominick Grift [this message]
2014-12-31 16:09 ` [refpolicy] [PATCH v2 2/6] Locate authdaemon socket and communicate with authdaemon Sven Vermeulen
2014-12-31 16:09 ` [refpolicy] [PATCH v2 3/6] Allow authdaemon to access selinux fs to check SELinux state Sven Vermeulen
2014-12-31 16:09 ` [refpolicy] [PATCH v2 4/6] Grant setuid/setgid to courier_pop_t Sven Vermeulen
2014-12-31 16:09 ` [refpolicy] [PATCH v2 5/6] Execute courier helper script after authentication Sven Vermeulen
2014-12-31 16:09 ` [refpolicy] [PATCH v2 6/6] Courier IMAP needs to manage the users' maildir Sven Vermeulen
2014-12-31 18:00 ` [refpolicy] [PATCH v2 0/6] Courier IMAPd related policy changes Dominick Grift
2015-01-01 18:29 ` Dominick Grift

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150101174121.GA4840@bigboy.network2 \
    --to=dac.override@gmail.com \
    --cc=refpolicy@oss.tresys.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.