Bugzilla #1106: Don't pass DKIM compound log line as format string
authorTom Kistner <tom@tahini.csx.cam.ac.uk>
Sat, 30 Apr 2011 12:20:17 +0000 (13:20 +0100)
committerTom Kistner <tom@tahini.csx.cam.ac.uk>
Sat, 30 Apr 2011 12:20:17 +0000 (13:20 +0100)
src/src/dkim.c

index e25ff8c85e9095d4d950030de3d42e4102ecdd11..2318cc394bef45e9ffda9b1784cb492943349fb6 100644 (file)
@@ -108,7 +108,7 @@ void dkim_exim_verify_finish(void) {
     /* Log a line for each signature */
     uschar *logmsg = string_append(NULL, &size, &ptr, 5,
 
-      string_sprintf( "DKIM: d=%s s=%s c=%s/%s a=%s ",
+      string_sprintf( "d=%s s=%s c=%s/%s a=%s ",
                       sig->domain,
                       sig->selector,
                       (sig->canon_headers == PDKIM_CANON_SIMPLE)?"simple":"relaxed",
@@ -176,7 +176,7 @@ void dkim_exim_verify_finish(void) {
     }
 
     logmsg[ptr] = '\0';
-    log_write(0, LOG_MAIN, (char *)logmsg);
+    log_write(0, LOG_MAIN, "DKIM: %s", logmsg);
 
     /* Build a colon-separated list of signing domains (and identities, if present) in dkim_signers */
     dkim_signers = string_append(dkim_signers,