linux-kernel - Re: [PATCH] LSM: Fix typo in a comment

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAHC9VhSLNNjXsW74bLbHX60Lco+Hjc6f4hp7Eb5hgC==_afPag@mail.gmail.com>
Date:   Wed, 5 Jul 2023 18:35:23 -0400
From:   Paul Moore <paul@...l-moore.com>
To:     Pairman Guo <pairmanxlr@...il.com>
Cc:     jmorris@...ei.org, serge@...lyn.com,
        linux-security-module@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] LSM: Fix typo in a comment

On Sun, Jul 2, 2023 at 1:09 PM Pairman Guo <pairmanxlr@...il.com> wrote:
>
> Hi maintainers,
>
> In the description of function definition security_file_lock(),
> the line "@cmd: fnctl command" has a typo where "fnctl" should be
> "fcntl". This patch fixes the typo.
>
> Please merge if it is the case. Thanks in advance.
>
> Signed-off-by: Pairman Guo <pairmanxlr@...il.com>
> ---
>  security/security.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

This looks good to me, I'll plan to merge this once the merge window
closes.  Thanks.

> diff --git a/security/security.c b/security/security.c
> index b720424ca37d..adaa7043c172 100644
> --- a/security/security.c
> +++ b/security/security.c
> @@ -2717,7 +2717,7 @@ int security_file_lock(struct file *file, unsigned int cmd)
>  /**
>   * security_file_fcntl() - Check if fcntl() op is allowed
>   * @file: file
> - * @cmd: fnctl command
> + * @cmd: fcntl command
>   * @arg: command argument
>   *
>   * Check permission before allowing the file operation specified by @cmd from
> --
> 2.41.0

-- 
paul-moore.com