All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dominick Grift <dominick.grift@defensec.nl>
To: Daniel Burgener <dburgener@linux.microsoft.com>
Cc: Russell Coker <russell@coker.com.au>, selinux-refpolicy@vger.kernel.org
Subject: Re: [PATCH] strict policy patches
Date: Mon, 11 Oct 2021 18:32:13 +0200	[thread overview]
Message-ID: <871r4rplo2.fsf@defensec.nl> (raw)
In-Reply-To: <a0ff06f4-4fd2-376c-e749-fbcf2677218c@linux.microsoft.com> (Daniel Burgener's message of "Mon, 11 Oct 2021 12:12:42 -0400")

Daniel Burgener <dburgener@linux.microsoft.com> writes:

>> ===================================================================
>> --- refpolicy-2.20210908.orig/policy/modules/system/systemd.te
>> +++ refpolicy-2.20210908/policy/modules/system/systemd.te
>> @@ -65,10 +65,6 @@ type systemd_activate_t;
>>   type systemd_activate_exec_t;
>>   init_system_domain(systemd_activate_t, systemd_activate_exec_t)
>>   -type systemd_analyze_t;
>> -type systemd_analyze_exec_t;
>> -init_daemon_domain(systemd_analyze_t, systemd_analyze_exec_t)
>> -
>>   type systemd_backlight_t;
>>   type systemd_backlight_exec_t;
>>   init_system_domain(systemd_backlight_t, systemd_backlight_exec_t)
>
> I proposed a similar change last year here and the consensus in the PR
> discussion was that it would make more sense to add policy for the 
> systemd_analyze_t domain for cases that wanted a transition there, but
> keeping the general approach of running in the parent domain.
>
> https://github.com/SELinuxProject/refpolicy/pull/321
>
> Of course, no one has actually submitted systemd_analyze_t policy yet,
> so maybe the demand for such a use case isn't all that high?
>
> -Daniel
>

I think I might have argued for keeping it around back then but I do not mind
removing it now. It certainly is not an init_daemon_domain(). One can
always add it later if needed.

-- 
gpg --locate-keys dominick.grift@defensec.nl
Key fingerprint = FCD2 3660 5D6B 9D27 7FC6  E0FF DA7E 521F 10F6 4098
Dominick Grift

  reply	other threads:[~2021-10-11 16:36 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-09 10:05 [PATCH] strict policy patches Russell Coker
2021-10-11 16:12 ` Daniel Burgener
2021-10-11 16:32   ` Dominick Grift [this message]
2021-10-27 13:09 ` Chris PeBenito

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=871r4rplo2.fsf@defensec.nl \
    --to=dominick.grift@defensec.nl \
    --cc=dburgener@linux.microsoft.com \
    --cc=russell@coker.com.au \
    --cc=selinux-refpolicy@vger.kernel.org \
    /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.