Re: [PATCH 02/12] utils: fix dead initialization

[Samuel Thibault]

Justus Winter, le Wed 20 Nov 2013 14:37:55 +0100, a écrit :
> Found using the Clang Static Analyzer.
> 
> * utils/frobauth.c (posix_parse_opt): Fix dead initialization.
> (no_ugids_parse_opt): Likewise.

Ack.

> ---
>  utils/frobauth.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/utils/frobauth.c b/utils/frobauth.c
> index ccb7c3b..44690d8 100644
> --- a/utils/frobauth.c
> +++ b/utils/frobauth.c
> @@ -209,7 +209,7 @@ ea_parse_opt (int key, char *arg, struct argp_state 
> *state)
>  static error_t
>  posix_parse_opt (int key, char *arg, struct argp_state *state)
>  {
> -  struct frobauth_argp_state *fs = state->hook;
> +  struct frobauth_argp_state *fs;
>  
>    switch (key)
>      {
> @@ -233,7 +233,7 @@ posix_parse_opt (int key, char *arg, struct argp_state 
> *state)
>  static error_t
>  no_ugids_parse_opt (int key, char *arg, struct argp_state *state)
>  {
> -  struct frobauth_argp_state *fs = state->hook;
> +  struct frobauth_argp_state *fs;
>  
>    switch (key)
>      {
> -- 
> 1.7.10.4
> 

-- 
Samuel
       La  fonction  memfrob() crypte les n premiers octets de la
       zone de mémoire  s  en  effectuant  un  OU-exclusif  entre
       chaque  octet  et le nombre 42.
(extrait de la page de man de memfrob -- Manuel du programmeur Linux)