Re: [PATCH] fix description of init_system_domain in init.if

["Christopher J. PeBenito" <cpebenito@tresys.com>]

On 04/27/11 10:22, Elia Pinto wrote:
> From: Elia Pinto <gitter.spiros@gmail.com>
> 
> Signed-off-by: Elia Pinto <gitter.spiros@gmail.com>
> ---
>  Changelog                     |    2 ++
>  policy/modules/system/init.if |    2 +-
>  2 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/Changelog b/Changelog
> index 99e8f45..e385097 100644
> --- a/Changelog
> +++ b/Changelog
> @@ -1,3 +1,5 @@
> +- Fix policy/modules/system/init.if init_system_domain description from 
> +  Elia Pinto.
>  - Postgresql selabel_lookup update from KaiGai Kohei.
>  - Dovecot managesieve support from Mika Pfluger.
>  - Semicolon after interface/template calls cleanup from Elia Pinto.

Please don't update the changelog.  It might have changed between the
time you created the patch and when it was committed.

> diff --git a/policy/modules/system/init.if b/policy/modules/system/init.if
> index cc83689..6553873 100644
> --- a/policy/modules/system/init.if
> +++ b/policy/modules/system/init.if
> @@ -304,7 +304,7 @@ interface(`init_ranged_daemon_domain',`
>  ## </summary>
>  ## <desc>
>  ##	<p>
> -##	Create a domain for long running processes (daemons/services)
> +##	Create a domain for short running processes (daemons/services)
>  ##	which are started by init scripts. These are generally applications that
>  ##	are used to initialize the system during boot.
>  ##	Long running processes

While you're right that it is for short running processes,
daemons/services aren't short running processes, so that part should be
removed.  I have manually fixed the description.

-- 
Chris PeBenito
Tresys Technology, LLC
www.tresys.com | oss.tresys.com

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.