Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: "José Pekkarinen" <jose.pekkarinen@unikie.com>
Cc: "Weber, Matthew L Collins" <Matthew.Weber@collins.com>,
	Adam Duskett <aduskett@gmail.com>,
	buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v2] package/libselinux: Add autorelabel for first boot
Date: Fri, 20 Aug 2021 21:16:56 +0200	[thread overview]
Message-ID: <20210820191656.GS27036@scaer> (raw)
In-Reply-To: <CAJPV9Mqd-c_0JD82KQU5hMH32UVMy71ded1Ht7vtEr6iJhw+xg@mail.gmail.com>

José, All,

+Matthew +Adam, our resident SELinux experts: questions for you toward
the end...

(resend as I acutally forgot to add them)

On 2021-08-20 15:19 +0300, José Pekkarinen spake thusly:
> On Fri, Aug 20, 2021 at 12:05 AM Yann E. MORIN < [1]yann.morin.1998@free.fr> wrote:
> > On 2021-08-19 12:29 +0300, José Pekkarinen spake thusly:
> > > Currently buildroot ship libselinux without triggering
> > > this option, which often shows inconsistencies between
> > > what the refpolicy defines as a label for a file and
> > > what the actual file has. Triggering an initial relabel
> > > would help activating enforcing state right away without
> > > requiring to enter it once in permissive and tweak the
> > > labels.
[--SNIP--]
> > Isn't this going to fail on read-only filesystems? Relabelling suposedly
> > requires that extended attributes be added/updated/removed, and that
> > requires a read-write filesystem...
> > Can't we do the re-labelling at the time we create the filesystem, i.e.
> > in fs/common.mk?
> > And it seems we already have that:
[--SNIP--]
> > So why is the labelling wrong? Can't we fix it right there rather than
> > at runtime?
> It's is not wrong, it was just unnoticed by my eyeballs,

:-)

> however, there is a case this is not covering properly and preventing
> the userspace to run right away in enforcing mode, because at
> this time not all files in /dev are populated, and running it in
> permissive mode multiple complains from selinux to the serial
> devices turn up. If you have some suggestions how we can
> improve this case, I'm happy to bring more changes.

What I understand from your explanations, above, is that we have to have
some labels (i.e. extended attributes) set on files in /dev, or the
policy may reference objects that are not properly labeled.

OK, so this hit the thick wall circling around my very limited knowledge
of how SELinux works.

Matthew, Adam, any help/explanations/details/review would be much welcome.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

  parent reply	other threads:[~2021-08-20 19:17 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-19  9:29 [Buildroot] [PATCH v2] package/libselinux: Add autorelabel for first boot José Pekkarinen
2021-08-19 21:05 ` Yann E. MORIN
2021-08-20 12:19   ` José Pekkarinen
2021-08-20 19:15     ` Yann E. MORIN
2021-08-20 19:16     ` Yann E. MORIN [this message]
2021-08-23  5:43       ` José Pekkarinen
2021-08-23 14:19       ` [Buildroot] [External] " Weber, Matthew L Collins via buildroot
2021-08-25 11:33         ` José Pekkarinen
2021-09-03  6:53           ` José Pekkarinen

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=20210820191656.GS27036@scaer \
    --to=yann.morin.1998@free.fr \
    --cc=Matthew.Weber@collins.com \
    --cc=aduskett@gmail.com \
    --cc=buildroot@buildroot.org \
    --cc=jose.pekkarinen@unikie.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox