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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8/30/21 12:44 AM, Adrian Huang wrote:
From: Adrian Huang <ahuang12@xxxxxxxxxx>

1. backwords -> backwards
2. Remove 'and' and whitespace
3. Correct the possessive form of "process"

Cc: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
Signed-off-by: Adrian Huang <ahuang12@xxxxxxxxxx>

Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>

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




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux