exec: remove useless comment

Commit Message

Askar Safin Dec. 27, 2023, 12:31 a.m. UTC

Function name is wrong and the comment tells us nothing
---
 fs/exec.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Kees Cook Jan. 8, 2024, 5:24 p.m. UTC | #1

On Wed, Dec 27, 2023 at 03:31:20AM +0300, Askar Safin wrote:
> Function name is wrong and the comment tells us nothing

Yes, quite true. :) Can you please include a Signed-off-by?
https://docs.kernel.org/process/submitting-patches.html#sign-your-work-the-developer-s-certificate-of-origin

-Kees

> ---
>  fs/exec.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/fs/exec.c b/fs/exec.c
> index 4aa19b24f..d882a6128 100644
> --- a/fs/exec.c
> +++ b/fs/exec.c
> @@ -1804,9 +1804,6 @@ static int exec_binprm(struct linux_binprm *bprm)
>  	return 0;
>  }
>  
> -/*
> - * sys_execve() executes a new program.
> - */
>  static int bprm_execve(struct linux_binprm *bprm,
>  		       int fd, struct filename *filename, int flags)
>  {
> -- 
> 2.11.0
>

Patch

diff --git a/fs/exec.c b/fs/exec.c
index 4aa19b24f..d882a6128 100644
--- a/fs/exec.c
+++ b/fs/exec.c
@@ -1804,9 +1804,6 @@  static int exec_binprm(struct linux_binprm *bprm)
 	return 0;
 }
 
-/*
- * sys_execve() executes a new program.
- */
 static int bprm_execve(struct linux_binprm *bprm,
 		       int fd, struct filename *filename, int flags)
 {