Remove attempts to quieten compiler static-checking (more)
authorJeremy Harris <jgh146exb@wizmail.org>
Sun, 28 Jun 2020 13:16:20 +0000 (14:16 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Sun, 28 Jun 2020 13:16:20 +0000 (14:16 +0100)
18 files changed:
src/src/acl.c
src/src/auths/auth-spa.c
src/src/dbfn.c
src/src/debug.c
src/src/dns.c
src/src/exim.c
src/src/exim_dbutil.c
src/src/filtertest.c
src/src/ip.c
src/src/log.c
src/src/match.c
src/src/routers/ipliteral.c
src/src/smtp_in.c
src/src/spool_mbox.c
src/src/tls-openssl.c
src/src/transport.c
src/src/transports/pipe.c
src/src/transports/smtp.c

index 1bb0cbb8aec35006963c64cd2258f565aa02c649..8e17a0e9c618ba0cc90d6dc605dd45c92935b915 100644 (file)
@@ -1186,8 +1186,6 @@ acl_verify_reverse(uschar **user_msgptr, uschar **log_msgptr)
 {
 int rc;
 
 {
 int rc;
 
-user_msgptr = user_msgptr;  /* stop compiler warning */
-
 /* Previous success */
 
 if (sender_host_name != NULL) return OK;
 /* Previous success */
 
 if (sender_host_name != NULL) return OK;
index 44c99e9f6a68d2fc7cda8043a92f98a2bbe8511c..583a2b955f318500048e083021fa18bafad18890 100644 (file)
@@ -1395,8 +1395,6 @@ int i;
 int p = (int)getpid();
 int random_seed = (int)time(NULL) ^ ((p << 16) | p);
 
 int p = (int)getpid();
 int random_seed = (int)time(NULL) ^ ((p << 16) | p);
 
-request = request;  /* Added by PH to stop compilers whinging */
-
 /* Ensure challenge data is cleared, in case it isn't all used. This
 patch added by PH on suggestion of Russell King */
 
 /* Ensure challenge data is cleared, in case it isn't all used. This
 patch added by PH on suggestion of Russell King */
 
index bbf20a1d5ba12ac602b7105b106142a0e2db35ae..aa083200a5784d7465f914459033c383518e1562 100644 (file)
@@ -423,7 +423,6 @@ dbfn_scan(open_db *dbblock, BOOL start, EXIM_CURSOR **cursor)
 {
 EXIM_DATUM key_datum, value_datum;
 uschar *yield;
 {
 EXIM_DATUM key_datum, value_datum;
 uschar *yield;
-value_datum = value_datum;    /* dummy; not all db libraries use this */
 
 DEBUG(D_hints_lookup) debug_printf_indent("dbfn_scan\n");
 
 
 DEBUG(D_hints_lookup) debug_printf_indent("dbfn_scan\n");
 
index 3a7d6a6f51b2e30b7abdbd412b623f9b476a02ac..44e0cee2e83bf98720169a68d000b21532afe17d 100644 (file)
@@ -332,7 +332,7 @@ if (fstat(fd, &s) == 0 && (s.st_mode & S_IFMT) == S_IFSOCK)
   socklen_t alen = sizeof(a);
   struct sockaddr_in * sinp = (struct sockaddr_in *)&a;
   struct sockaddr_in6 * sin6p = (struct sockaddr_in6 *)&a;
   socklen_t alen = sizeof(a);
   struct sockaddr_in * sinp = (struct sockaddr_in *)&a;
   struct sockaddr_in6 * sin6p = (struct sockaddr_in6 *)&a;
-  struct sockaddr_un * sa_unp ; (struct sockaddr_un *)&a;
+  struct sockaddr_un * sa_unp = (struct sockaddr_un *)&a;
 
   if (getsockname(fd, &a, &alen) == 0)
     switch (sinp->sin_family)
 
   if (getsockname(fd, &a, &alen) == 0)
     switch (sinp->sin_family)
index b567c3e71813b87fdfdf5667e8c8a32a2f73f8b2..98c44b9f29225553941db47bcd5d93e33add3f46 100644 (file)
@@ -334,7 +334,6 @@ char * trace = NULL;
 #ifdef rr_trace
 # define TRACE DEBUG(D_dns)
 #else
 #ifdef rr_trace
 # define TRACE DEBUG(D_dns)
 #else
-trace = trace;
 # define TRACE if (FALSE)
 #endif
 
 # define TRACE if (FALSE)
 #endif
 
@@ -1249,7 +1248,7 @@ switch (type)
 
        /* Extract the numerical SRV fields (p is incremented) */
        GETSHORT(priority, p);
 
        /* Extract the numerical SRV fields (p is incremented) */
        GETSHORT(priority, p);
-       GETSHORT(weight, p);    weight = weight; /* compiler quietening */
+       GETSHORT(weight, p);
        GETSHORT(port, p);
 
        /* Check the CSA version number */
        GETSHORT(port, p);
 
        /* Check the CSA version number */
index dcfc066232e3c8b833e632f1315bbb06134d29c3..9d3d456ef3f80c8db619e977efda15e6cfd2b95c 100644 (file)
@@ -53,7 +53,7 @@ return store_get((int)size, TRUE);
 }
 
 static void
 }
 
 static void
-function_dummy_free(void *block) { block = block; }
+function_dummy_free(void * block) {}
 
 static void *
 function_store_malloc(size_t size)
 
 static void *
 function_store_malloc(size_t size)
@@ -62,7 +62,7 @@ return store_malloc((int)size);
 }
 
 static void
 }
 
 static void
-function_store_free(void *block)
+function_store_free(void * block)
 {
 store_free(block);
 }
 {
 store_free(block);
 }
@@ -4101,10 +4101,8 @@ privilege by now. Before the chdir, we try to ensure that the directory exists.
 
 if (Uchdir(spool_directory) != 0)
   {
 
 if (Uchdir(spool_directory) != 0)
   {
-  int dummy;
-  (void)directory_make(spool_directory, US"", SPOOL_DIRECTORY_MODE, FALSE);
-  dummy = /* quieten compiler */ Uchdir(spool_directory);
-  dummy = dummy;       /* yet more compiler quietening, sigh */
+  (void) directory_make(spool_directory, US"", SPOOL_DIRECTORY_MODE, FALSE);
+  (void) Uchdir(spool_directory);
   }
 
 /* Handle calls with the -bi option. This is a sendmail option to rebuild *the*
   }
 
 /* Handle calls with the -bi option. This is a sendmail option to rebuild *the*
@@ -4115,7 +4113,7 @@ script. */
 
 if (bi_option)
   {
 
 if (bi_option)
   {
-  (void)fclose(config_file);
+  (void) fclose(config_file);
   if (bi_command && *bi_command)
     {
     int i = 0;
   if (bi_command && *bi_command)
     {
     int i = 0;
index 8513bc0c7a3dc8756212604ba2ee473e715d4698..742952758b3a7ec2f3b3b99b360056ff123615b3 100644 (file)
@@ -515,7 +515,6 @@ dbfn_scan(open_db *dbblock, BOOL start, EXIM_CURSOR **cursor)
 {
 EXIM_DATUM key_datum, value_datum;
 uschar *yield;
 {
 EXIM_DATUM key_datum, value_datum;
 uschar *yield;
-value_datum = value_datum;    /* dummy; not all db libraries use this */
 
 /* Some dbm require an initialization */
 
 
 /* Some dbm require an initialization */
 
index f54cbefb5582e80d950c2f3052ed0958a4a59cd8..a2a60a8dd8a5be996aa58bfbb025b634fdee5ca0 100644 (file)
@@ -95,7 +95,7 @@ if (!dot_ended && !feof(stdin))
       if (s > message_body_end + message_body_visible) s = message_body_end;
       message_size++;
       }
       if (s > message_body_end + message_body_visible) s = message_body_end;
       message_size++;
       }
-    READ_END: ch = ch;  /* Some compilers don't like null statements */
+    READ_END: ;
     }
   if (s == message_body_end || s[-1] != '\n') body_linecount++;
   }
     }
   if (s == message_body_end || s[-1] != '\n') body_linecount++;
   }
index a5958e95e43098556d6c7c7d94ef84756616ae45..c09962a38933667e6ff10e04131a1e799f53dd20 100644 (file)
@@ -127,8 +127,6 @@ if (af == AF_INET6)
   return sizeof(sin->v6);
   }
 else
   return sizeof(sin->v6);
   }
 else
-#else     /* HAVE_IPv6 */
-af = af;  /* Avoid compiler warning */
 #endif    /* HAVE_IPV6 */
 
 /* Setup code when using IPv4 socket. The wildcard address is "". */
 #endif    /* HAVE_IPV6 */
 
 /* Setup code when using IPv4 socket. The wildcard address is "". */
@@ -209,8 +207,6 @@ if (af == AF_INET6)
   s_len = sizeof(s_in6);
   }
 else
   s_len = sizeof(s_in6);
   }
 else
-#else     /* HAVE_IPV6 */
-af = af;  /* Avoid compiler warning */
 #endif    /* HAVE_IPV6 */
 
 /* For an IPv4 address, use an IPv4 sockaddr structure, even on a system with
 #endif    /* HAVE_IPV6 */
 
 /* For an IPv4 address, use an IPv4 sockaddr structure, even on a system with
index 99eba5f905971c5c80bfb6b8e65ae92aa08e9a85..9f3f45e2a2c41525d375f61d979a8415b1f4fb6e 100644 (file)
@@ -1188,10 +1188,7 @@ if (flags & LOG_PANIC)
     panic_recurseflag = FALSE;
 
     if (panic_save_buffer)
     panic_recurseflag = FALSE;
 
     if (panic_save_buffer)
-      {
-      int i = write(paniclogfd, panic_save_buffer, Ustrlen(panic_save_buffer));
-      i = i;   /* compiler quietening */
-      }
+      (void) write(paniclogfd, panic_save_buffer, Ustrlen(panic_save_buffer));
 
     written_len = write_to_fd_buf(paniclogfd, g->s, g->ptr);
     if (written_len != g->ptr)
 
     written_len = write_to_fd_buf(paniclogfd, g->s, g->ptr);
     if (written_len != g->ptr)
index 2f6c922510483ed1b310caec94fece34c948f84a..18de9779641a11138e42c72e3014ff3d3b516535 100644 (file)
@@ -104,8 +104,6 @@ uschar *filename = NULL;
 uschar *keyquery, *result, *semicolon;
 void *handle;
 
 uschar *keyquery, *result, *semicolon;
 void *handle;
 
-error = error;  /* Keep clever compilers from complaining */
-
 if (valueptr) *valueptr = NULL;
 
 /* For regular expressions, use cb->origsubject rather than cb->subject so that
 if (valueptr) *valueptr = NULL;
 
 /* For regular expressions, use cb->origsubject rather than cb->subject so that
@@ -1004,8 +1002,6 @@ uschar *subject = cb->address;
 const uschar *s;
 uschar *pdomain, *sdomain;
 
 const uschar *s;
 uschar *pdomain, *sdomain;
 
-error = error;  /* Keep clever compilers from complaining */
-
 DEBUG(D_lists) debug_printf("address match test: subject=%s pattern=%s\n",
   subject, pattern);
 
 DEBUG(D_lists) debug_printf("address match test: subject=%s pattern=%s\n",
   subject, pattern);
 
index f0b3b193f1398b8954d05a7c1e40fb68d4854f0e..09775ed83220829c172c5039dd8bc4432ce200d6 100644 (file)
@@ -57,7 +57,6 @@ ipliteral_router_init(router_instance *rblock)
 ipliteral_router_options_block *ob =
   (ipliteral_router_options_block *)(rblock->options_block);
 */
 ipliteral_router_options_block *ob =
   (ipliteral_router_options_block *)(rblock->options_block);
 */
-rblock = rblock;
 }
 
 
 }
 
 
index 39f51999433118dd3dacf11fb2f67c27ad3ca1a4..ef3964f5c6720d385107f289a410c8f38336c4d0 100644 (file)
@@ -4497,10 +4497,8 @@ while (done <= 0)
 # endif
       else
 #endif
 # endif
       else
 #endif
+       (void) fwrite(g->s, 1, g->ptr, smtp_out);
 
 
-       {
-       int i = fwrite(g->s, 1, g->ptr, smtp_out); i = i; /* compiler quietening */
-       }
       DEBUG(D_receive)
        {
        uschar *cr;
       DEBUG(D_receive)
        {
        uschar *cr;
index 4f3c4d25bb26de9a6db786685927843131cb05d7..9f1efbf59f41353fc6a1d0ef9775916886377eb8 100644 (file)
@@ -226,12 +226,11 @@ if (spool_mbox_ok && !f.no_mbox_unspool)
   for (struct dirent *entry; entry = readdir(tempdir); )
     {
     uschar *name = US entry->d_name;
   for (struct dirent *entry; entry = readdir(tempdir); )
     {
     uschar *name = US entry->d_name;
-    int dummy;
     if (Ustrcmp(name, US".") == 0 || Ustrcmp(name, US"..") == 0) continue;
 
     file_path = string_sprintf("%s/%s", mbox_path, name);
     debug_printf("unspool_mbox(): unlinking '%s'\n", file_path);
     if (Ustrcmp(name, US".") == 0 || Ustrcmp(name, US"..") == 0) continue;
 
     file_path = string_sprintf("%s/%s", mbox_path, name);
     debug_printf("unspool_mbox(): unlinking '%s'\n", file_path);
-    dummy = unlink(CS file_path); dummy = dummy;       /* compiler quietening */
+    (void) unlink(CS file_path);
     }
 
   closedir(tempdir);
     }
 
   closedir(tempdir);
index 525afd65078dd66f6b321dc89ae7700d5acad2cd..50384cbd565e3bb2dd9694c407954b059649f2ba 100644 (file)
@@ -496,7 +496,6 @@ RSA *rsa_key;
 BIGNUM *bn = BN_new();
 #endif
 
 BIGNUM *bn = BN_new();
 #endif
 
-export = export;     /* Shut picky compilers up */
 DEBUG(D_tls) debug_printf("Generating %d bit RSA key...\n", keylength);
 
 #ifdef EXIM_HAVE_RSA_GENKEY_EX
 DEBUG(D_tls) debug_printf("Generating %d bit RSA key...\n", keylength);
 
 #ifdef EXIM_HAVE_RSA_GENKEY_EX
index f9ff521f78f5208429650d12f337e0f6345bb43c..aed743d6237d5ade007634d1c3fa7fefe03bdbdf 100644 (file)
@@ -1400,7 +1400,6 @@ if (write_pid > 0)
        int dummy = read(pfd[pipe_read], (void *)&save_errno, sizeof(int));
         dummy = read(pfd[pipe_read], (void *)&tctx->addr->more_errno, sizeof(int));
         dummy = read(pfd[pipe_read], (void *)&tctx->addr->delivery_time, sizeof(struct timeval));
        int dummy = read(pfd[pipe_read], (void *)&save_errno, sizeof(int));
         dummy = read(pfd[pipe_read], (void *)&tctx->addr->more_errno, sizeof(int));
         dummy = read(pfd[pipe_read], (void *)&tctx->addr->delivery_time, sizeof(struct timeval));
-       dummy = dummy;          /* compiler quietening */
         yield = FALSE;
         }
       }
         yield = FALSE;
         }
       }
index 27422bd4275253faff52eec52ed1039d011d3382..1cb574ee72c791a4e98b2699774b930e9b18fd62 100644 (file)
@@ -144,13 +144,6 @@ pipe_transport_setup(transport_instance *tblock, address_item *addrlist,
 pipe_transport_options_block *ob =
   (pipe_transport_options_block *)(tblock->options_block);
 
 pipe_transport_options_block *ob =
   (pipe_transport_options_block *)(tblock->options_block);
 
-addrlist = addrlist;  /* Keep compiler happy */
-dummy = dummy;
-uid = uid;
-gid = gid;
-errmsg = errmsg;
-ob = ob;
-
 #ifdef HAVE_SETCLASSRESOURCES
 if (ob->use_classresources)
   {
 #ifdef HAVE_SETCLASSRESOURCES
 if (ob->use_classresources)
   {
index 0f3ade9fd58e91b6c0c39e867b14548c4e2bb331..46663b0523dda55c1c0adea503eef8e2d32f7731 100644 (file)
@@ -3428,7 +3428,6 @@ uschar *message = NULL;
 uschar new_message_id[MESSAGE_ID_LENGTH + 1];
 smtp_context * sx = store_get(sizeof(*sx), TRUE);      /* tainted, for the data buffers */
 
 uschar new_message_id[MESSAGE_ID_LENGTH + 1];
 smtp_context * sx = store_get(sizeof(*sx), TRUE);      /* tainted, for the data buffers */
 
-suppress_tls = suppress_tls;  /* stop compiler warning when no TLS support */
 *message_defer = FALSE;
 
 memset(sx, 0, sizeof(*sx));
 *message_defer = FALSE;
 
 memset(sx, 0, sizeof(*sx));