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

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

[Adrian Huang]

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>
---
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)
 {
-- 
2.27.0

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

[Randy Dunlap]

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