Re: [PATCH] remove stray `;` after function definitions

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

 



applied, thanks. Surprised the fairly aggressive set of -W flags we use
by default didn't catch this...

On Tue, 26 Nov 2019, Michael Forney wrote:

> This is not valid in ISO C.
> ---
>  ssh-sk.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/ssh-sk.c b/ssh-sk.c
> index 2b25c42f..3d6ac3d0 100644
> --- a/ssh-sk.c
> +++ b/ssh-sk.c
> @@ -153,7 +153,7 @@ sshsk_free_enroll_response(struct sk_enroll_response *r)
>  	freezero(r->signature, r->signature_len);
>  	freezero(r->attestation_cert, r->attestation_cert_len);
>  	freezero(r, sizeof(*r));
> -};
> +}
>  
>  static void
>  sshsk_free_sign_response(struct sk_sign_response *r)
> @@ -163,7 +163,7 @@ sshsk_free_sign_response(struct sk_sign_response *r)
>  	freezero(r->sig_r, r->sig_r_len);
>  	freezero(r->sig_s, r->sig_s_len);
>  	freezero(r, sizeof(*r));
> -};
> +}
>  
>  #ifdef WITH_OPENSSL
>  /* Assemble key from response */
> -- 
> 2.24.0
> 
> _______________________________________________
> openssh-unix-dev mailing list
> openssh-unix-dev@xxxxxxxxxxx
> https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev
> 
_______________________________________________
openssh-unix-dev mailing list
openssh-unix-dev@xxxxxxxxxxx
https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev



[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux