Fix compiling problems with OpenSSL 0.9.8e.
authorPhilip Hazel <ph10@hermes.cam.ac.uk>
Tue, 13 Mar 2007 09:50:22 +0000 (09:50 +0000)
committerPhilip Hazel <ph10@hermes.cam.ac.uk>
Tue, 13 Mar 2007 09:50:22 +0000 (09:50 +0000)
doc/doc-txt/ChangeLog
src/ACKNOWLEDGMENTS
src/src/tls-openssl.c

index 5aac30f4996354bfa995ce417659f2059f07fb0c..55a1aaa4c9dfc9ae4fc36272e5906e91909a51f4 100644 (file)
@@ -1,4 +1,4 @@
-$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.487 2007/03/01 14:06:56 ph10 Exp $
+$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.488 2007/03/13 09:50:22 ph10 Exp $
 
 Change log file for Exim from version 4.21
 -------------------------------------------
 
 Change log file for Exim from version 4.21
 -------------------------------------------
@@ -141,6 +141,14 @@ PH/32 Change the Dovecot authenticator to use read() and write() on the socket
       instead of the C I/O that was originally supplied, because problems were
       reported on Solaris.
 
       instead of the C I/O that was originally supplied, because problems were
       reported on Solaris.
 
+PH/33 Compile failed with OpenSSL 0.9.8e. This was due to a coding error in
+      Exim which did not show up earlier: it was assuming that a call to
+      SSL_CTX_set_info_callback() might give an error value. In fact, there is
+      no error. In previous releases of OpenSSL, SSL_CTX_set_info_callback()
+      was a macro that became an assignment, so it seemed to work. This has
+      changed to a proper function call with a void return, hence the compile
+      error. Exim's code has been fixed.
+
 
 Exim version 4.66
 -----------------
 
 Exim version 4.66
 -----------------
index 99ace355439d7e463dc188a9a592a5c2be360a8f..4172a5b0e3ebc99e091803bb23d08e96bcdb822c 100644 (file)
@@ -1,4 +1,4 @@
-$Cambridge: exim/src/ACKNOWLEDGMENTS,v 1.74 2007/03/01 14:06:56 ph10 Exp $
+$Cambridge: exim/src/ACKNOWLEDGMENTS,v 1.75 2007/03/13 09:50:22 ph10 Exp $
 
 EXIM ACKNOWLEDGEMENTS
 
 
 EXIM ACKNOWLEDGEMENTS
 
@@ -20,7 +20,7 @@ relatively small patches.
 Philip Hazel
 
 Lists created: 20 November 2002
 Philip Hazel
 
 Lists created: 20 November 2002
-Last updated:  01 March 2007
+Last updated:  13 March 2007
 
 
 THE OLD LIST
 
 
 THE OLD LIST
@@ -162,6 +162,7 @@ Steve Haslam              Lots of stuff, including
                             HMAC computations
                             Better error messages for BDB
 Sheldon Hearn             Suggested patch for smtp_accept_max_nonmail_hosts
                             HMAC computations
                             Better error messages for BDB
 Sheldon Hearn             Suggested patch for smtp_accept_max_nonmail_hosts
+                          Fix for compile error with OpenSSL 0.9.8e
 Bryan Henderson           Patch to use RM_COMMAND everywhere during building
 Jakob Hirsch              Patch for % operator
                           Patch for arbitrarily named ACL variables
 Bryan Henderson           Patch to use RM_COMMAND everywhere during building
 Jakob Hirsch              Patch for % operator
                           Patch for arbitrarily named ACL variables
index 1ae725b865107cb5d8a90bb4e55a4b07228e8e9c..85f4e15fec3808c0d28cdafa1fb9c010f310b784 100644 (file)
@@ -1,4 +1,4 @@
-/* $Cambridge: exim/src/src/tls-openssl.c,v 1.10 2007/01/18 15:35:42 ph10 Exp $ */
+/* $Cambridge: exim/src/src/tls-openssl.c,v 1.11 2007/03/13 09:50:22 ph10 Exp $ */
 
 /*************************************************
 *     Exim - an Internet mail transport agent    *
 
 /*************************************************
 *     Exim - an Internet mail transport agent    *
@@ -343,8 +343,7 @@ if (!RAND_status())
 /* Set up the information callback, which outputs if debugging is at a suitable
 level. */
 
 /* Set up the information callback, which outputs if debugging is at a suitable
 level. */
 
-if (!(SSL_CTX_set_info_callback(ctx, (void (*)())info_callback)))
-  return tls_error(US"SSL_CTX_set_info_callback", host);
+SSL_CTX_set_info_callback(ctx, (void (*)())info_callback);
 
 /* The following patch was supplied by Robert Roselius */
 
 
 /* The following patch was supplied by Robert Roselius */