[exim-dev] [Bug 934] HAVE_SENDFILE breaks DKIM

Top Page
Delete this message
Reply to this message
Author: Git Commit
Date:  
To: exim-dev
Old-Topics: [exim-dev] [Bug 934] New: HAVE_SENDFILE breaks DKIM
Subject: [exim-dev] [Bug 934] HAVE_SENDFILE breaks DKIM
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=934




--- Comment #3 from Git Commit <git@???> 2014-06-11 20:17:13 ---
Git commit:
http://git.exim.org/exim.git/commitdiff/75febd1bb37ede1b350b9e45e02f18b7d0067824

commit 75febd1bb37ede1b350b9e45e02f18b7d0067824
Author:     Wolfgang Breyha <wbreyha@???>
AuthorDate: Wed Jun 11 19:19:49 2014 +0100
Commit:     Jeremy Harris <jgh146exb@???>
CommitDate: Wed Jun 11 19:58:21 2014 +0100


    Fix dkim for no-key case under SENDFILE compile.  Bug 934


    Tested-by: <wbreyha@???>
---
 src/src/transport.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)


diff --git a/src/src/transport.c b/src/src/transport.c
index d222adc..3648bfc 100644
--- a/src/src/transport.c
+++ b/src/src/transport.c
@@ -1068,7 +1068,7 @@ if (dkim_private_key && dkim_domain && dkim_selector)
       uschar *dkim_strict_result = expand_string(dkim_strict);
       if (dkim_strict_result)
        if ( (strcmpic(dkim_strict,US"1") == 0) ||
-            (strcmpic(dkim_strict,US"true") == 0) )
+            (strcmpic(dkim_strict,US"true") == 0) ) 
          {
          /* Set errno to something halfway meaningful */
          save_errno = EACCES;
@@ -1104,8 +1104,8 @@ if (dkim_private_key && dkim_domain && dkim_selector)
     }
   }


- /* Fetch file positition (the size) */
- size = lseek(dkim_fd,0,SEEK_CUR);
+/* Fetch file size */
+size = lseek(dkim_fd, 0, SEEK_END);

/* Rewind file */
lseek(dkim_fd, 0, SEEK_SET);


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email