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 > -- Kees Cook