tidying

diff --git a/src/src/host.c b/src/src/host.c
index 6a6e7abf8bcb061c4b326f44e9f622198332a181..b65168c68a1870bd5270b0377a38b6ce4fbad9e5 100644 (file)
--- a/src/src/host.c
+++ b/src/src/host.c
@@ -1521,7 +1521,7 @@ int len;
 uschar *s, *t;
 struct hostent *hosts;
 struct in_addr addr;
-unsigned long time_msec;
+unsigned long time_msec = 0;   /* init to quieten dumb static analysis */
 
 if (slow_lookup_log) time_msec = get_time_in_ms();
 

diff --git a/src/src/mime.c b/src/src/mime.c
index 7b1fa387d7cde8be790e8c2c7d5b498fcdfb0142..6a64b26b1f8f6e8a17ff645029f0740130726a45 100644 (file)
--- a/src/src/mime.c
+++ b/src/src/mime.c
@@ -12,7 +12,16 @@
 
 FILE *mime_stream = NULL;
 uschar *mime_current_boundary = NULL;
-static int mime_header_list_size = sizeof(mime_header_list)/sizeof(mime_header);
+
+static mime_header mime_header_list[] = {
+  { US"content-type:",              13, &mime_content_type },
+  { US"content-disposition:",       20, &mime_content_disposition },
+  { US"content-transfer-encoding:", 26, &mime_content_transfer_encoding },
+  { US"content-id:",                11, &mime_content_id },
+  { US"content-description:",       20, &mime_content_description }
+};
+
+static int mime_header_list_size = nelem(mime_header_list);
 
 static mime_parameter mime_parameter_list[] = {
   { US"name=",     5, &mime_filename },

diff --git a/src/src/mime.h b/src/src/mime.h
index 5c030ebcaf70a7c9a6e77e1126137c8a243391ba..8dfdf55547defcc2e6f98bc8e2c0568e808811c4 100644 (file)
--- a/src/src/mime.h
+++ b/src/src/mime.h
@@ -27,16 +27,6 @@ typedef struct mime_header {
   uschar ** value;
 } mime_header;
 
-static mime_header mime_header_list[] = {
-  { US"content-type:",              13, &mime_content_type },
-  { US"content-disposition:",       20, &mime_content_disposition },
-  { US"content-transfer-encoding:", 26, &mime_content_transfer_encoding },
-  { US"content-id:",                11, &mime_content_id },
-  { US"content-description:",       20, &mime_content_description }
-};
-
-
-
 
 typedef struct mime_parameter {
   uschar *  name;

diff --git a/src/src/moan.c b/src/src/moan.c
index 7d1a2c681d2b557c62177f362fc3038216ebca37..3a490028ba01dee153eaf552916165a0573caccf 100644 (file)
--- a/src/src/moan.c
+++ b/src/src/moan.c
@@ -279,14 +279,12 @@ if (bounce_return_message)
 
   if (bounce_return_body && message_file)
     {
-    int ch;
-    enum {midline, beginline, haddot} state = beginline;
     BOOL enddot = dot_ends && message_file == stdin;
     uschar * buf = store_get(bounce_return_linesize_limit+2);
 
     if (firstline) fprintf(f, "%s", CS firstline);
 
-    while (fgets(buf, bounce_return_linesize_limit+2, message_file))
+    while (fgets(CS buf, bounce_return_linesize_limit+2, message_file))
       {
       int len;
 
@@ -306,11 +304,11 @@ if (bounce_return_message)
       if (size_limit > 0 && len > size_limit - written)
        {
        buf[size_limit - written] = '\0';
-       fputs(buf, f);
+       fputs(CS buf, f);
        break;
        }
 
-      fputs(buf, f);
+      fputs(CS buf, f);
       }
     }
 #ifdef EXPERIMENTAL_DMARC

diff --git a/src/src/readconf.c b/src/src/readconf.c
index 1119651f0a27b484c90ebfd59d1d63762c2510a1..7396c8bd930f7c83694bdf262b5ac1f2dc9ae3a6 100644 (file)
--- a/src/src/readconf.c
+++ b/src/src/readconf.c
@@ -2709,15 +2709,13 @@ if (type == NULL)
     if (environ)
       {
       uschar **p;
-      size_t n;
       for (p = USS environ; *p; p++) ;
-      n = p - USS environ;
       qsort(environ, p - USS environ, sizeof(*p), string_compare_by_pointer);
 
       for (p = USS environ; *p; p++)
         {
         if (no_labels) *(Ustrchr(*p, '=')) = '\0';
-        puts(*p);
+        puts(CS *p);
         }
       }
     return;

diff --git a/src/src/tls-openssl.c b/src/src/tls-openssl.c
index 83594c9731db8e7da88916a2d4232f1018ac09fb..ca687a6326f7e88333594b76f350e388f6276bea 100644 (file)
--- a/src/src/tls-openssl.c
+++ b/src/src/tls-openssl.c
@@ -266,6 +266,7 @@ DEBUG(D_tls) debug_printf("Generating %d bit RSA key...\n", keylength);
 
 #ifdef EXIM_HAVE_RSA_GENKEY_EX
 if (  !BN_set_word(bn, (unsigned long)RSA_F4)
+   || !(rsa_key = RSA_new())
    || !RSA_generate_key_ex(rsa_key, keylength, bn, NULL)
    )
 #else