Correct logging bug for failures to get hints database read lock.
authorPhilip Hazel <ph10@hermes.cam.ac.uk>
Thu, 20 Apr 2006 15:05:15 +0000 (15:05 +0000)
committerPhilip Hazel <ph10@hermes.cam.ac.uk>
Thu, 20 Apr 2006 15:05:15 +0000 (15:05 +0000)
doc/doc-txt/ChangeLog
src/src/dbfn.c

index 8fb90d75007ccf3680638645231267d23ce6a80a..28044a2798105614fd56edbf1a90c6df8297c258 100644 (file)
@@ -1,4 +1,4 @@
-$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.344 2006/04/20 14:11:29 ph10 Exp $
+$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.345 2006/04/20 15:05:15 ph10 Exp $
 
 Change log file for Exim from version 4.21
 -------------------------------------------
@@ -33,6 +33,9 @@ PH/05 Batch deliveries by appendfile and pipe transports did not work when the
       File deliveries just didn't batch; pipe deliveries might have suffered
       odd errors.
 
+PH/06 A failure to get a lock for a hints database would erroneously always say
+      "Failed to get write lock", even when it was really a read lock.
+
 
 Exim version 4.61
 -----------------
index 938400ccbf3decc5381d5e0880b1b976e0fa0227..cefa0b323d50d0da037fe46d62ce43f5bbae8f90 100644 (file)
@@ -1,4 +1,4 @@
-/* $Cambridge: exim/src/src/dbfn.c,v 1.8 2006/02/07 11:19:00 ph10 Exp $ */
+/* $Cambridge: exim/src/src/dbfn.c,v 1.9 2006/04/20 15:05:15 ph10 Exp $ */
 
 /*************************************************
 *     Exim - an Internet mail transport agent    *
@@ -143,7 +143,7 @@ if (sigalrm_seen) errno = ETIMEDOUT;
 if (rc < 0)
   {
   log_write(0, LOG_MAIN, "Failed to get %s lock for %s: %s",
-    ((flags & O_RDONLY) != 0)? "read" : "write", buffer,
+    read_only? "read" : "write", buffer,
     (errno == ETIMEDOUT)? "timed out" : strerror(errno));
   (void)close(dbblock->lockfd);
   errno = 0;       /* Indicates locking failure */