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

[Karol Lewandowski <lmctlx@gmail.com>]

On Fri, Oct 31, 2014 at 05:36:33PM +0100, Karol Lewandowski wrote:
> Signed-off-by: Karol Lewandowski <k.lewandowsk@samsung.com>
> ---
>  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