New LSM hooks

[Casey Schaufler]

On 2/6/2019 9:06 AM, Stephen Smalley wrote:
> On 2/6/19 11:30 AM, Casey Schaufler wrote:
>> On 2/5/2019 5:11 PM, James Morris wrote:
>>> On Tue, 5 Feb 2019, Paul Moore wrote:
>>>
>>>> I believe that will always be a problem, no matter what we do.  The
>>>> point I was trying to make was that everyone, especially the
>>>> maintainers, need to watch for this when patches are posted and make
>>>> sure the patch author posts to the LSM list in addition to any of the
>>>> relevant LSM specific lists.
>>> Right, and there is no way a new LSM hook should ever be added to the
>>> kernel without review and ack/signoffs from folks on the LSM list
>>> (especially those who are maintainers of in-tree LSMs).
>>>
>>> Casey, do you have any examples of this happening?
>>
>> overlayfs (according to my records - which may be flawed)
>> is a prime example. Inifiniband hooks were reviewed/acked
>> for SELinux, but there was never an attempt made to work
>> with other security module maintainers. Yes, they were posted
>> to LSM, but under the title "SELinux support for Infiniband".
>
> overlayfs hooks were also posted to and discussed on lsm list, including comments from you.  Admittedly the cover patch said Overlayfs SELinux Support but the individual patches for the hooks were "security, overlayfs: provide copy up security hook for unioned files" and "security,overlayfs: Provide security hook for copy up of xattrs for overlay file".

OK, apologies all around. My records are still better
than my memory, but neither is perfect.

>
> In either case, did you request a change that was ignored?

Smack support was definitely not included. I started
getting complaints almost immediately when overlayfs
hit upstream.