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@gmail.com>
---
 security/security.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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

On Sun, Jul 2, 2023 at 1:09 PM Pairman Guo <pairmanxlr@gmail.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@gmail.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

On Wed, Jul 5, 2023 at 6:35 PM Paul Moore <paul@paul-moore.com> wrote:
> On Sun, Jul 2, 2023 at 1:09 PM Pairman Guo <pairmanxlr@gmail.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@gmail.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.

I'm merging this into lsm/next now, thanks!

-- 
paul-moore.com