Re: [PATCH v2 1/1] exec: fix typo and grammar mistake in comment

[Randy Dunlap <rdunlap@infradead.org>]

On 8/30/21 12:44 AM, Adrian Huang wrote:
> From: Adrian Huang <ahuang12@lenovo.com>
> 
> 1. backwords -> backwards
> 2. Remove 'and' and whitespace
> 3. Correct the possessive form of "process"
> 
> Cc: Randy Dunlap <rdunlap@infradead.org>
> Signed-off-by: Adrian Huang <ahuang12@lenovo.com>

Acked-by: Randy Dunlap <rdunlap@infradead.org>

Thanks.

> ---
> Changes since v2:
>   * Correct possessive form of "process" and fix the grammar, per Randy
> 
>   fs/exec.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/fs/exec.c b/fs/exec.c
> index 38f63451b928..d0e20fedde21 100644
> --- a/fs/exec.c
> +++ b/fs/exec.c
> @@ -504,7 +504,7 @@ static int bprm_stack_limits(struct linux_binprm *bprm)
>   
>   /*
>    * 'copy_strings()' copies argument/environment strings from the old
> - * processes's memory to the new process's stack.  The call to get_user_pages()
> + * process's memory to the new process's stack. The call to get_user_pages()
>    * ensures the destination page is created and not swapped out.
>    */
>   static int copy_strings(int argc, struct user_arg_ptr argv,
> @@ -533,7 +533,7 @@ static int copy_strings(int argc, struct user_arg_ptr argv,
>   		if (!valid_arg_len(bprm, len))
>   			goto out;
>   
> -		/* We're going to work our way backwords. */
> +		/* We're going to work our way backwards. */
>   		pos = bprm->p;
>   		str += len;
>   		bprm->p -= len;
> @@ -600,7 +600,7 @@ static int copy_strings(int argc, struct user_arg_ptr argv,
>   }
>   
>   /*
> - * Copy and argument/environment string from the kernel to the processes stack.
> + * Copy an argument/environment string from the kernel to the process's stack.
>    */
>   int copy_string_kernel(const char *arg, struct linux_binprm *bprm)
>   {
> 


-- 
~Randy