[exim-cvs] Harmonised TLS library version reporting.

Top Page
Delete this message
Reply to this message
Author: Exim Git Commits Mailing List
Date:  
To: exim-cvs
Subject: [exim-cvs] Harmonised TLS library version reporting.
Gitweb: http://git.exim.org/exim.git/commitdiff/754a0503134b184183f64c04ed30a3524fc3860b
Commit:     754a0503134b184183f64c04ed30a3524fc3860b
Parent:     109066729a54f6ba5c4e8bc174133da33242e930
Author:     Phil Pennock <pdp@???>
AuthorDate: Tue Mar 22 07:01:52 2011 -0400
Committer:  Phil Pennock <pdp@???>
CommitDate: Tue Mar 22 07:01:52 2011 -0400


    Harmonised TLS library version reporting.


    Only show if debugging.
    Layout now matches that introduced for other libraries in 4.74 PP/03.
---
 doc/doc-txt/ChangeLog |    3 +++
 src/src/exim.c        |   12 ++++--------
 src/src/tls-gnu.c     |    6 ++++--
 src/src/tls-openssl.c |    6 ++++--
 4 files changed, 15 insertions(+), 12 deletions(-)


diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog
index 94952af..8200163 100644
--- 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 3592f30..30974c9 100644
--- 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 8fcdd10..23f8bd6 100644
--- 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 b6a8dcb..e9628ba 100644
--- 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));
 }