Re: [PATCH 2/2] audit: remove unnecessary #if CONFIG_AUDIT

From: Andrew Morton
Date: Mon Apr 15 2013 - 17:02:39 EST


On Fri, 12 Apr 2013 17:34:21 +0800 Gao feng <gaofeng@xxxxxxxxxxxxxx> wrote:

> The files include kernel/audit.h are complied only
> when CONFIG_AUDIT is set.
>
> Just like audit_pid,there is no need to surround
> audit_ever_enabled with CONFIG_AUDIT.
>
> Signed-off-by: Gao feng <gaofeng@xxxxxxxxxxxxxx>
> ---
> kernel/audit.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/kernel/audit.h b/kernel/audit.h
> index d06ffc1..11468d9 100644
> --- a/kernel/audit.h
> +++ b/kernel/audit.h
> @@ -59,9 +59,7 @@ struct audit_entry {
> struct audit_krule rule;
> };
>
> -#ifdef CONFIG_AUDIT
> extern int audit_ever_enabled;
> -#endif
>
> extern int audit_pid;

Well actually there's no need to put the ifdefs around such an extern
declaration at all. The one small downside to omitting the ifdefs is
that a programming mistake may then be revealed at link-time instead of
at compile-time. But we very often omit the ifdefs, because reducing visual
clutter is more important.


--
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/