Re: [PATCH 08/11] linux-user/syscall: Remove unused variable from execve

[Laurent Vivier]

Le 12/07/2021 à 23:55, Richard Henderson a écrit :
> From clang-13:
> linux-user/syscall.c:8503:17: error: variable 'total_size' set but not used \
>     [-Werror,-Wunused-but-set-variable]
> 
> Cc: Laurent Vivier <laurent@vivier.eu>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>  linux-user/syscall.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
> index 2e826206d2..c27debba8c 100644
> --- a/linux-user/syscall.c
> +++ b/linux-user/syscall.c
> @@ -8500,7 +8500,6 @@ static abi_long do_syscall1(void *cpu_env, int num, 
> abi_long arg1,
>              abi_ulong guest_envp;
>              abi_ulong addr;
>              char **q;
> -            int total_size = 0;
>  
>              argc = 0;
>              guest_argp = arg2;
> @@ -8532,7 +8531,6 @@ static abi_long do_syscall1(void *cpu_env, int num, 
> abi_long arg1,
>                      break;
>                  if (!(*q = lock_user_string(addr)))
>                      goto execve_efault;
> -                total_size += strlen(*q) + 1;
>              }
>              *q = NULL;
>  
> @@ -8544,7 +8542,6 @@ static abi_long do_syscall1(void *cpu_env, int num, 
> abi_long arg1,
>                      break;
>                  if (!(*q = lock_user_string(addr)))
>                      goto execve_efault;
> -                total_size += strlen(*q) + 1;
>              }
>              *q = NULL;
>  
> 

Acked-by: Laurent Vivier <laurent@vivier.eu>