On Wed, 2023-10-18 at 12:35 -0400, Paul Moore wrote:

On Wed, Oct 18, 2023 at 10:15 AM Roberto Sassu
[off-list ref] wrote:

quoted

On 10/18/2023 3:09 PM, Mimi Zohar wrote:

...

quoted

quoted

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