Re: [PATCH 1/5] kdbus: extend structures with security pointer for lsm

From: Karol Lewandowski
Date: Sun Nov 16 2014 - 20:47:43 EST


On Fri, Oct 31, 2014 at 05:36:33PM +0100, Karol Lewandowski wrote:
> Signed-off-by: Karol Lewandowski <k.lewandowsk@xxxxxxxxxxx>
> ---
> drivers/misc/kdbus/bus.h | 2 ++
> drivers/misc/kdbus/connection.h | 2 ++
> drivers/misc/kdbus/domain.h | 2 ++
> 3 files changed, 6 insertions(+)
>
> diff --git a/drivers/misc/kdbus/bus.h b/drivers/misc/kdbus/bus.h
> index fd9d843..5c403ef 100644
> --- a/drivers/misc/kdbus/bus.h
> +++ b/drivers/misc/kdbus/bus.h
> @@ -49,6 +49,7 @@
> * @conn_hash: Map of connection IDs
> * @monitors_list: Connections that monitor this bus
> * @meta: Meta information about the bus creator
> + * @security: LSM security blob
> *
> * A bus provides a "bus" endpoint / device node.
> *
> @@ -84,6 +85,7 @@ struct kdbus_bus {
> struct list_head monitors_list;
>
> struct kdbus_meta *meta;
> + void *security;

One minor note - with possibility of LSM modules trying to access
these structures it would be worth to have kdbus headers in, say
include/linux/kdbus or similar globally accessible location.

Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/