Re: [PATCH] LSM: lsm_hooks.h - fix missing colon in docstring

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Mar 25, 2019 at 4:05 AM Ondrej Mosnacek <omosnace@xxxxxxxxxx> wrote:
> Apparently without it it is incorrect syntax and causes a warning about
> undocumented struct field.
>
> Fixes: b230d5aba2d1 ("LSM: add new hook for kernfs node initialization")
> Reported-by: kbuild test robot <lkp@xxxxxxxxx>
> Signed-off-by: Ondrej Mosnacek <omosnace@xxxxxxxxxx>
> ---
>  include/linux/lsm_hooks.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

If it makes you feel any better, I didn't realize the colon was that
important either.  Thanks for the fix, merged into selinux/next.

> diff --git a/include/linux/lsm_hooks.h b/include/linux/lsm_hooks.h
> index 0dd5bda719e6..b987dc282d63 100644
> --- a/include/linux/lsm_hooks.h
> +++ b/include/linux/lsm_hooks.h
> @@ -447,7 +447,7 @@
>   *
>   * Security hooks for kernfs node operations
>   *
> - * @kernfs_init_security
> + * @kernfs_init_security:
>   *     Initialize the security context of a newly created kernfs node based
>   *     on its own and its parent's attributes.
>   *
> --
> 2.20.1

-- 
paul moore
www.paul-moore.com



[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux