Harmonised TLS library version reporting.

Only show if debugging.
Layout now matches that introduced for other libraries in 4.74 PP/03.

diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog
index 94952afa5bf09e28624fe21ab0a31eabf8f53834..820016310e3c443d768965722e30bb0c09fbb08e 100644 (file)
--- a/doc/doc-txt/ChangeLog
+++ b/doc/doc-txt/ChangeLog
@@ -8,6 +8,9 @@ Exim version 4.76
 
 PP/01 The new ldap_require_cert option would segfault if used.  Fixed.
 
+PP/02 Harmonised TLS library version reporting; only show if debugging.
+      Layout now matches that introduced for other libraries in 4.74 PP/03.
+
 
 Exim version 4.75
 -----------------

diff --git a/src/src/exim.c b/src/src/exim.c
index 3592f30ddce6ddc412dea92ef065c4e462eb353f..30974c9c6621e6bd0fffe015125dc57ac98c3dc3 100644 (file)
--- a/src/src/exim.c
+++ b/src/src/exim.c
@@ -912,14 +912,6 @@ if (fixed_never_users[0] > 0)
 
 fprintf(f, "Size of off_t: " SIZE_T_FMT "\n", sizeof(off_t));
 
-/* This runtime check is to help diagnose library linkage mismatches which
-result in segfaults and the like; as such, it's left until the end,
-just in case.  There will still be a "Configuration file is" line still to
-come. */
-#ifdef SUPPORT_TLS
-tls_version_report(f);
-#endif
-
 /* Everything else is details which are only worth reporting when debugging.
 Perhaps the tls_version_report should move into this too. */
 DEBUG(D_any) do {
@@ -941,6 +933,10 @@ DEBUG(D_any) do {
   fprintf(f, "Compiler: <unknown>\n");
 #endif
 
+#ifdef SUPPORT_TLS
+  tls_version_report(f);
+#endif
+
 #ifdef AUTH_CYRUS_SASL
   auth_cyrus_sasl_version_report(f);
 #endif

diff --git a/src/src/tls-gnu.c b/src/src/tls-gnu.c
index 8fcdd109dd7ecec7a4d99690e079b30d598e74a1..23f8bd6adb01815b338254c8967bcac49a60d05d 100644 (file)
--- a/src/src/tls-gnu.c
+++ b/src/src/tls-gnu.c
@@ -1326,8 +1326,10 @@ Returns:     nothing
 void
 tls_version_report(FILE *f)
 {
-fprintf(f, "GnuTLS compile-time version: %s\n", LIBGNUTLS_VERSION);
-fprintf(f, "GnuTLS runtime version: %s\n", gnutls_check_version(NULL));
+fprintf(f, "Library version: GnuTLS: Compile: %s\n"
+           "                         Runtime: %s\n",
+           LIBGNUTLS_VERSION,
+           gnutls_check_version(NULL));
 }
 
 /* End of tls-gnu.c */

diff --git a/src/src/tls-openssl.c b/src/src/tls-openssl.c
index b6a8dcb8c74e085ca4f17f0378665e43c1d79f1e..e9628ba29a1e50488a836b20b9ab4d3e0da8fb08 100644 (file)
--- a/src/src/tls-openssl.c
+++ b/src/src/tls-openssl.c
@@ -1081,8 +1081,10 @@ Returns:     nothing
 void
 tls_version_report(FILE *f)
 {
-fprintf(f, "OpenSSL compile-time version: %s\n", OPENSSL_VERSION_TEXT);
-fprintf(f, "OpenSSL runtime version: %s\n", SSLeay_version(SSLEAY_VERSION));
+fprintf(f, "Library version: OpenSSL: Compile: %s\n"
+           "                          Runtime: %s\n",
+           OPENSSL_VERSION_TEXT,
+           SSLeay_version(SSLEAY_VERSION));
 }