linux-kernel - Re: [PATCH] netlink: Remove the include of files doesn't exist

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAEjxPJ7C=fq3uM0t_uQJ00ZZbieMP5ZO3BXMVY7FzB0ENJU1Nw@mail.gmail.com>
Date: Fri, 5 Apr 2024 12:03:48 -0400
From: Stephen Smalley <stephen.smalley.work@...il.com>
To: I Hsin Cheng <richard120310@...il.com>
Cc: akpm@...ux-foundation.org, paul@...l-moore.com, eparis@...isplace.org, 
	selinux@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] netlink: Remove the include of files doesn't exist

On Fri, Apr 5, 2024 at 10:19 AM I Hsin Cheng <richard120310@...il.com> wrote:
>
> The file for /security/selinux/security.h and the file for
> /include/linux/selinux_netlink.h are no long exist. However the
> preprocessor still performs the inclusion of those files, these
> behaviors should be removed to ensure it's bug-free.
>
> Signed-off-by: I Hsin Cheng <richard120310@...il.com>

NAK. You are incorrect on both counts and evidently didn't try compiling.
At a purely source level, I agree it might look confusing, but the
Makefile adds the security/selinux/include directory (where security.h
lives) to the include path and selinux_netlink.h lives under
include/uapi/linux which is auto-magically included.

> ---
>  security/selinux/netlink.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/security/selinux/netlink.c b/security/selinux/netlink.c
> index 1760aee71..8ce36abaa 100644
> --- a/security/selinux/netlink.c
> +++ b/security/selinux/netlink.c
> @@ -13,11 +13,9 @@
>  #include <linux/kernel.h>
>  #include <linux/export.h>
>  #include <linux/skbuff.h>
> -#include <linux/selinux_netlink.h>
>  #include <net/net_namespace.h>
>  #include <net/netlink.h>
>
> -#include "security.h"
>
>  static struct sock *selnl __ro_after_init;
>
> --
> 2.34.1
>