>>>>> "Wietse" == Wietse Venema <wie...@porcupine.org> writes:

Wietse> John Stoffel:
>> >>>>> "Viktor" == Viktor Dukhovni <postfix-us...@dukhovni.org> writes:
>> 
Viktor> On an mostly unrelated note, OpenSSL 3.0 (~Q4 2020) is changing the
Viktor> error API, so we'll eventually need:
>> 
Viktor> --- src/tls/tls_misc.c
Viktor> +++ src/tls/tls_misc.c
Viktor> @@ -1332,6 +1332,18 @@ void    tls_print_errors(void)
Viktor> int     line;
Viktor> int     flags;
>> 
Viktor> +#if defined(OPENSSL_VERSION_PREREQ) && OPENSSL_VERSION_PREREQ(3,0)
Viktor> +    const char *func;
Viktor> +
Viktor> +    while ((err = ERR_get_error_all(&file, &line, &func, &data, 
&flags)) != 0) {
Viktor> +       ERR_error_string_n(err, buffer, sizeof(buffer));
Viktor> +       if (flags & ERR_TXT_STRING)
Viktor> +           msg_warn("TLS library problem: %s:%s:%s:%d:%s:",
Viktor> +                    buffer, file, func, line, data);
Viktor> +       else
Viktor> +           msg_warn("TLS library problem: %s:%s:%s:%d:", buffer, file, 
func, line);
>> 
>> Can we be more specific here with what the problem is?  Maybe the
>> 'data' entry has more details that I'm missing here.  

Wietse> You mean, eliminate the 'flags & ERR_TXT_STRING' test?

More the msg_warn("TLS library problem: ...") is what I'm commenting
on, but I suspect that the needed info is in the 'data' string passed
in for error reporting.

Thanks to both you and Victor for all you've done with postfix,
Excellent software!

Reply via email to