Re: [PATCH v15 00/11] LSM: Three basic syscalls
From: Mimi Zohar
Date: Wed Oct 18 2023 - 16:23:10 EST
On Wed, 2023-10-18 at 12:35 -0400, Paul Moore wrote:
> On Wed, Oct 18, 2023 at 10:15 AM Roberto Sassu
> <roberto.sassu@xxxxxxxxxxxxxxx> wrote:
> > On 10/18/2023 3:09 PM, Mimi Zohar wrote:
>
> ...
>
> > > I agree with Roberto. All three should be defined: LSM_ID_INTEGRITY,
> > > LSM_ID_IMA, LSM_ID_EVM.
> >
> > I did not try yet, but the 'integrity' LSM does not need an LSM ID. With
> > the last version adding hooks to 'ima' or 'evm', it should be sufficient
> > to keep DEFINE_LSM(integrity) with the request to store a pointer in the
> > security blob (even the init function can be a dummy function).
>
> First off, this *really* should have been brought up way, way, *way*
> before now. This patchset has been discussed for months, and bringing
> up concerns in the eleventh hour is borderline rude.
As everyone knows IMA and EVM are not LSMs at this point.
So the only thing that is "rude" is the way you're responding in this
thread.
>
> At least we haven't shipped this in a tagged release from Linus yet,
> so there is that.
What does that have to do with anything?! Code changes.
Mimi