Re: [PATCH] audit: Modifying indentation issue.

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

 



On Fri, Jul 29, 2022 at 5:05 AM l00581214 <liutie4@xxxxxxxxxx> wrote:
>
> From: Tie Liu <liutie4@xxxxxxxxxx>
>
> Modifying indentation issue in audit_rate_check().
>
> Signed-off-by: Tie Liu <liutie4@xxxxxxxxxx>
> ---
>  kernel/audit.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

While I agree that the style in this patch is preferable to the
existing style in the code, I'm growing weary of these minor style
change from developers who have not contributing any other meaningful
change to the audit subsystem.  Once you have established a history of
useful contributions to the audit subsystem feel free to (re)submit
this patch.

> diff --git a/kernel/audit.c b/kernel/audit.c
> index 7690c29d4..deeede166 100644
> --- a/kernel/audit.c
> +++ b/kernel/audit.c
> @@ -324,7 +324,8 @@ static inline int audit_rate_check(void)
>         unsigned long           elapsed;
>         int                     retval     = 0;
>
> -       if (!audit_rate_limit) return 1;
> +       if (!audit_rate_limit)
> +               return 1;
>
>         spin_lock_irqsave(&lock, flags);
>         if (++messages < audit_rate_limit) {

-- 
paul-moore.com



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux