On Fri, Dec 02, 2022 at 09:41:01AM +0800, Wang Yufen wrote:
> Fix to return a negative error code from create_elf_fdpic_tables()
> instead of 0.
>
> Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
> Signed-off-by: Wang Yufen <wangyufen@xxxxxxxxxx>
Thanks for the catch! Yeah, it looks like this has been wrong for a long
time. :)
-Kees
> ---
> fs/binfmt_elf_fdpic.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/fs/binfmt_elf_fdpic.c b/fs/binfmt_elf_fdpic.c
> index 08d0c87..9ce5e1f 100644
> --- a/fs/binfmt_elf_fdpic.c
> +++ b/fs/binfmt_elf_fdpic.c
> @@ -434,8 +434,9 @@ static int load_elf_fdpic_binary(struct linux_binprm *bprm)
> current->mm->start_stack = current->mm->start_brk + stack_size;
> #endif
>
> - if (create_elf_fdpic_tables(bprm, current->mm,
> - &exec_params, &interp_params) < 0)
> + retval = create_elf_fdpic_tables(bprm, current->mm, &exec_params,
> + &interp_params);
> + if (retval < 0)
> goto error;
>
> kdebug("- start_code %lx", current->mm->start_code);
> --
> 1.8.3.1
>
--
Kees Cook
