Hi, On Wed, Aug 17, 2022 at 01:03:19PM +0200, Lev Stipakov wrote: > From: Lev Stipakov <[email protected]> > > The correct return type for those functions is int, > so adjust return type accordingly for the cases > when DCO is not defined.
The change is fine, but you only go halfway...
> ---
> src/openvpn/dco.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/openvpn/dco.h b/src/openvpn/dco.h
> index 6b5c016a..04556471 100644
> --- a/src/openvpn/dco.h
> +++ b/src/openvpn/dco.h
> @@ -292,7 +292,7 @@ dco_update_keys(dco_context_t *dco, struct tls_multi
> *multi)
> ASSERT(false);
> }
>
> -static inline bool
> +static inline int
> dco_p2p_add_new_peer(struct context *c)
> {
> return true;
Here, you change bool->int, but leave "return true"...
> @@ -310,10 +310,10 @@ dco_remove_peer(struct context *c)
> {
> }
>
> -static inline bool
> +static inline int
> dco_multi_add_new_peer(struct multi_context *m, struct multi_instance *mi)
> {
> - return true;
> + return 0;
> }
... here, you also change the return to "int".
Can we have this consistently? ;-)
gert
--
"If was one thing all people took for granted, was conviction that if you
feed honest figures into a computer, honest figures come out. Never doubted
it myself till I met a computer with a sense of humor."
Robert A. Heinlein, The Moon is a Harsh Mistress
Gert Doering - Munich, Germany [email protected]
signature.asc
Description: PGP signature
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
