silence various compiler complaints; expose NVALGRIND

diff --git a/src/src/EDITME b/src/src/EDITME
index c4aedb6bd982109cfdbd9be70db4e0eac64547fa..f247f44a9a981626d9ed6abf79a4a68641958493 100644 (file)
--- a/src/src/EDITME
+++ b/src/src/EDITME
@@ -1198,6 +1198,12 @@ TMPDIR="/tmp"
 # PERL_LIBS=
 
 
 # PERL_LIBS=
 
 

+#------------------------------------------------------------------------------
+# If you wish to disable valgrind in the binary, define NVALGRIND=1.
+# This should not be needed.
+
+# NVALGRIND=1
+

 #------------------------------------------------------------------------------
 # Identifying the daemon: When an Exim daemon starts up, it writes its pid
 # (process id) to a file so that it can easily be identified. The path of the
 #------------------------------------------------------------------------------
 # Identifying the daemon: When an Exim daemon starts up, it writes its pid
 # (process id) to a file so that it can easily be identified. The path of the

diff --git a/src/src/acl.c b/src/src/acl.c
index 50d5d85941e7c1107d3082c8b0d148a1d0b1d4f2..3cafd818419c8e4e490f03bce92046e136967414 100644 (file)
--- a/src/src/acl.c
+++ b/src/src/acl.c
@@ -992,7 +992,7 @@ for (p = q = hstring; *p != 0; )
     if (*s == ':' || !isgraph(*s)) break;
     }
 
     if (*s == ':' || !isgraph(*s)) break;
     }
 

-  s = string_sprintf("%s%.*s", (*s == ':')? "" : "X-ACL-Warn: ", q - p, p);
+  s = string_sprintf("%s%.*s", (*s == ':')? "" : "X-ACL-Warn: ", (int) (q - p), p);

   hlen = Ustrlen(s);
 
   /* See if this line has already been added */
   hlen = Ustrlen(s);
 
   /* See if this line has already been added */


@@ -1723,16 +1723,16 @@ while ((ss = string_nextinlist(&list, &sep, big_buffer, big_buffer_size))
               != NULL)
           {
          callout_opt_t * op;
               != NULL)
           {
          callout_opt_t * op;

-         double period;
+         double period = 1.0F;

 
          for (op= callout_opt_list; op->name; op++)
 
          for (op= callout_opt_list; op->name; op++)

-           if (strncmpic(opt, op->name, strlen(op->name)) == 0)
+           if (strncmpic(opt, op->name, Ustrlen(op->name)) == 0)

              break;
 
          verify_options |= op->flag;
          if (op->has_option)
            {
              break;
 
          verify_options |= op->flag;
          if (op->has_option)
            {

-           opt += strlen(op->name);
+           opt += Ustrlen(op->name);

             while (isspace(*opt)) opt++;
             if (*opt++ != '=')
               {
             while (isspace(*opt)) opt++;
             if (*opt++ != '=')
               {


@@ -2093,7 +2093,7 @@ uschar buffer[STRING_SPRINTF_BUFFER_SIZE];
 va_start(ap, format);
 if (!string_vformat(buffer, sizeof(buffer), format, ap))
   log_write(0, LOG_MAIN|LOG_PANIC_DIE,
 va_start(ap, format);
 if (!string_vformat(buffer, sizeof(buffer), format, ap))
   log_write(0, LOG_MAIN|LOG_PANIC_DIE,

-    "string_sprintf expansion was longer than %d", sizeof(buffer));
+    "string_sprintf expansion was longer than %ld", sizeof(buffer));

 va_end(ap);
 *log_msgptr = string_sprintf(
   "error in arguments to \"ratelimit\" condition: %s", buffer);
 va_end(ap);
 *log_msgptr = string_sprintf(
   "error in arguments to \"ratelimit\" condition: %s", buffer);


@@ -2228,7 +2228,7 @@ while ((ss = string_nextinlist(&arg, &sep, big_buffer, big_buffer_size))
   else if (strcmpic(ss, US"per_addr") == 0)
     {
     RATE_SET(mode, PER_RCPT);
   else if (strcmpic(ss, US"per_addr") == 0)
     {
     RATE_SET(mode, PER_RCPT);

-    if (where != ACL_WHERE_RCPT) badacl = TRUE, unique = "*";
+    if (where != ACL_WHERE_RCPT) badacl = TRUE, unique = US"*";

       else unique = string_sprintf("%s@%s", deliver_localpart, deliver_domain);
     }
   else if (strncmpic(ss, US"count=", 6) == 0)
       else unique = string_sprintf("%s@%s", deliver_localpart, deliver_domain);
     }
   else if (strncmpic(ss, US"count=", 6) == 0)

diff --git a/src/src/auths/cram_md5.c b/src/src/auths/cram_md5.c
index 31de7c2d2e606139e137954d167d0ce019238994..e93d0384540e3b9c8c2f35dd64c3538d841bb893 100644 (file)
--- a/src/src/auths/cram_md5.c
+++ b/src/src/auths/cram_md5.c
@@ -153,8 +153,8 @@ auth_cram_md5_server(auth_instance *ablock, uschar *data)
 {
 auth_cram_md5_options_block *ob =
   (auth_cram_md5_options_block *)(ablock->options_block);
 {
 auth_cram_md5_options_block *ob =
   (auth_cram_md5_options_block *)(ablock->options_block);

-uschar *challenge = string_sprintf("<%d.%d@%s>", getpid(), time(NULL),
-  primary_hostname);
+uschar *challenge = string_sprintf("<%d.%ld@%s>", getpid(),
+    (long int) time(NULL), primary_hostname);

 uschar *clear, *secret;
 uschar digest[16];
 int i, rc, len;
 uschar *clear, *secret;
 uschar digest[16];
 int i, rc, len;

diff --git a/src/src/auths/heimdal_gssapi.c b/src/src/auths/heimdal_gssapi.c
index e01789e31e49b67179006e53270178c60dc3a298..c6e973ad9af80c613eb961803666e50d2ecbe3eb 100644 (file)
--- a/src/src/auths/heimdal_gssapi.c
+++ b/src/src/auths/heimdal_gssapi.c
@@ -526,7 +526,7 @@ exim_gssapi_error_defer(uschar *store_reset_point,
   va_start(ap, format);
   if (!string_vformat(buffer, sizeof(buffer), format, ap))
     log_write(0, LOG_MAIN|LOG_PANIC_DIE,
   va_start(ap, format);
   if (!string_vformat(buffer, sizeof(buffer), format, ap))
     log_write(0, LOG_MAIN|LOG_PANIC_DIE,

-        "exim_gssapi_error_defer expansion larger than %d",
+        "exim_gssapi_error_defer expansion larger than %lu",

         sizeof(buffer));
   va_end(ap);
 
         sizeof(buffer));
   va_end(ap);
 

diff --git a/src/src/config.h.defaults b/src/src/config.h.defaults
index f3e3d880a393774c34b4f9efe52ee218dcc3dbcd..c082b9269c6bf74e68334c708ba0bcc3d4107f4a 100644 (file)
--- a/src/src/config.h.defaults
+++ b/src/src/config.h.defaults
@@ -101,6 +101,8 @@ it's a default value. */
 #define MAX_NAMED_LIST               16
 #define MSGLOG_DIRECTORY_MODE      0750
 
 #define MAX_NAMED_LIST               16
 #define MSGLOG_DIRECTORY_MODE      0750
 

+#define NVALGRIND
+

 #define PID_FILE_PATH
 
 #define RADIUS_CONFIG_FILE
 #define PID_FILE_PATH
 
 #define RADIUS_CONFIG_FILE

diff --git a/src/src/store.c b/src/src/store.c
index 1d1458d0fade1afa8608e5ea46e157f8f17869b4..0a5a11fc2d4b439bc1c87a754dde3ba68a341af9 100644 (file)
--- a/src/src/store.c
+++ b/src/src/store.c
@@ -39,6 +39,9 @@ The following different types of store are recognized:
 
 
 #include "exim.h"
 
 
 #include "exim.h"

+/* keep config.h before memcheck.h, for NVALGRIND */
+#include "config.h"
+

 #include "memcheck.h"
 
 
 #include "memcheck.h"
 
 


@@ -342,7 +345,7 @@ if ((char *)ptr < bc || (char *)ptr > bc + b->length)
     if ((char *)ptr >= bc && (char *)ptr <= bc + b->length) break;
     }
   if (b == NULL)
     if ((char *)ptr >= bc && (char *)ptr <= bc + b->length) break;
     }
   if (b == NULL)

-    log_write(0, LOG_MAIN|LOG_PANIC_DIE, "internal error: store_reset(%d) "
+    log_write(0, LOG_MAIN|LOG_PANIC_DIE, "internal error: store_reset(%p) "

       "failed: pool=%d %-14s %4d", ptr, store_pool, filename, linenumber);
   }
 
       "failed: pool=%d %-14s %4d", ptr, store_pool, filename, linenumber);
   }
 

diff --git a/src/src/transports/tf_maildir.c b/src/src/transports/tf_maildir.c
index 66760d1b0f4287f7a6fcb5759927eac433dec102..8e57d28b2041777fb47a14573a30b75f1cb5aeb6 100644 (file)
--- a/src/src/transports/tf_maildir.c
+++ b/src/src/transports/tf_maildir.c
@@ -555,7 +555,7 @@ else
 
   (void)gettimeofday(&tv, NULL);
   tempname = string_sprintf("%s/tmp/%lu.H%luP%lu.%s", path, tv.tv_sec,
 
   (void)gettimeofday(&tv, NULL);
   tempname = string_sprintf("%s/tmp/%lu.H%luP%lu.%s", path, tv.tv_sec,

-    tv.tv_usec, getpid(), primary_hostname);
+    tv.tv_usec, (long unsigned) getpid(), primary_hostname);

 
   fd = Uopen(tempname, O_RDWR|O_CREAT|O_EXCL, ob->mode ? ob->mode : 0600);
   if (fd >= 0)
 
   fd = Uopen(tempname, O_RDWR|O_CREAT|O_EXCL, ob->mode ? ob->mode : 0600);
   if (fd >= 0)