All of lore.kernel.org
 help / color / mirror / Atom feed
From: sven.vermeulen@siphos.be (Sven Vermeulen)
To: refpolicy@oss.tresys.com
Subject: [refpolicy] [PATCH 1/2] Asterisk admin must be able to run 'asterisk -r'
Date: Tue, 27 Sep 2011 18:35:04 +0200	[thread overview]
Message-ID: <20110927163503.GA17946@siphos.be> (raw)
In-Reply-To: <4E81C982.7000507@redhat.com>

On Tue, Sep 27, 2011 at 09:02:58AM -0400, Daniel J Walsh wrote:
> An asterisk admin should not be running the application in his his own
> context, he should be allowed to restart it in the asterisk_t domain
> which is why we have asterisk_initrc_exec_t. And are moving towards
> using asterisk_systemctl() for systemd controls.

Having stream_connect rights doesn't allow him to run the application in his
own domain. What this patch offers is to allow the asterisk admin to run
"asterisk -r" so he can get the CLI open. The CLI doesn't work if asterisk
isn't running, and restarting it doesn't refresh the process (so it remains
running in asterisk_t), it is more of a reload.

The use of "asterisk -r" is often done by asterisk admins. We've asked a few
of them to check if this is really necessary and they all agreed that having
"asterisk -r" possible is a major requirement for asterisk administration. 

The use of the init script is indeed still necessary for stop and start
operations as well as some general management activities, but administering
asterisk is more than just those operational activities.

Also, to reply to Dominick's suggestion to only have the
asterisk_stream_connect for the asterisk admin, that won't work since he
would still need can_exec rights on the asterisk_exec_t. Also, since he is
asterisk admin, he'll need asterisk_admin() anyhow.

Wkr,
	Sven Vermeulen

  reply	other threads:[~2011-09-27 16:35 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-26 19:55 [refpolicy] [PATCH 0/2] Asterisk updates Sven Vermeulen
2011-09-26 19:57 ` [refpolicy] [PATCH 1/2] Asterisk admin must be able to run 'asterisk -r' Sven Vermeulen
2011-09-26 20:23   ` Dominick Grift
2011-09-27 16:39     ` Sven Vermeulen
2011-09-27 17:02       ` Dominick Grift
2011-09-27 17:28         ` Sven Vermeulen
2011-09-27 13:02   ` Daniel J Walsh
2011-09-27 16:35     ` Sven Vermeulen [this message]
2011-09-27 16:55       ` Dominick Grift
2011-09-26 19:57 ` [refpolicy] [PATCH 2/2] Make sysadm an asterisk administrator Sven Vermeulen
2011-09-26 20:12   ` 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=20110927163503.GA17946@siphos.be \
    --to=sven.vermeulen@siphos.be \
    --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.