LSM: Fix typo in a comment

Message ID 20230702170857.15603-1-pairmanxlr@gmail.com
State New
Headers
Series LSM: Fix typo in a comment |

Commit Message

Pairman Guo July 2, 2023, 5:08 p.m. UTC
  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(-)
  

Comments

Paul Moore July 5, 2023, 10:35 p.m. UTC | #1
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 July 10, 2023, 6:02 p.m. UTC | #2
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!
  

Patch

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