[RFC PATCH] lsm,selinux: pass the family information along with xfrm flow
Paul Moore
paul at paul-moore.com
Wed Sep 30 13:44:02 UTC 2020
On Tue, Sep 29, 2020 at 7:09 PM James Morris <jmorris at namei.org> wrote:
> I'm not keen on adding a parameter which nobody is using. Perhaps a note
> in the header instead?
On Wed, Sep 30, 2020 at 6:14 AM Herbert Xu <herbert at gondor.apana.org.au> wrote:
> Please at least change to the struct flowi to flowi_common if we're
> not adding a family field.
It did feel a bit weird adding a (currently) unused parameter, so I
can understand the concern, I just worry that a comment in the code
will be easily overlooked. I also thought about passing a pointer to
the nested flowi_common struct, but it doesn't appear that this is
done anywhere else in the stack so it felt wrong to do it here.
--
paul moore
www.paul-moore.com
More information about the Linux-security-module-archive
mailing list