Hi, On 10-10-2020 10:14, Gert Doering wrote: > commit f0734e49956217 simplified key_state_export_keying_material(), > changing the function prototype. For older mbedTLS versions, there > is an "always fail" dummy function which was overlooked in that change. > > Fix prototype. > > v2: also adjust function return (NULL -> false) > > Signed-off-by: Gert Doering <[email protected]> > --- > src/openvpn/ssl_mbedtls.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/src/openvpn/ssl_mbedtls.c b/src/openvpn/ssl_mbedtls.c > index bb5633b7..11fbeae4 100644 > --- a/src/openvpn/ssl_mbedtls.c > +++ b/src/openvpn/ssl_mbedtls.c > @@ -252,14 +252,13 @@ key_state_export_keying_material(struct tls_session > *session, > } > } > #else > -unsigned char* > +bool > key_state_export_keying_material(struct tls_session *session, > const char* label, size_t label_size, > - size_t ekm_size, > - struct gc_arena *gc) > + void *ekm, size_t ekm_size) > { > /* Dummy function to avoid ifdefs in the common code */ > - return NULL; > + return false; > } > #endif /* HAVE_EXPORT_KEYING_MATERIAL */ > >
Sorry, totally missed this. Fix looks good. Acked-by: Steffan Karger <[email protected]> -Steffan _______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
