tidying
authorJeremy Harris <jgh146exb@wizmail.org>
Sun, 29 Sep 2019 14:55:16 +0000 (15:55 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Mon, 30 Sep 2019 15:50:54 +0000 (16:50 +0100)
src/src/functions.h
test/src/client.c

index 0cd0a0d18d5c1c493ed6aa7912fc3ce1d26c2cbe..31c0057736b4ab7e91fbd1dbb0637f78f0cf2041 100644 (file)
@@ -892,7 +892,7 @@ return string_sprintf("%s/%s/%s/%s/%s%s",
        spool_directory, queue_name, purpose, subdir, fname, suffix);
 }
 
        spool_directory, queue_name, purpose, subdir, fname, suffix);
 }
 
-static void
+static inline void
 set_subdir_str(uschar * subdir_str, const uschar * name,
        int search_sequence)
 {
 set_subdir_str(uschar * subdir_str, const uschar * name,
        int search_sequence)
 {
index d56a33f4001a7be3a7f3779d0cae6169e6c6189c..1cec2cb922ebf1159c7f783b17f29e695b8cb509 100644 (file)
@@ -569,76 +569,79 @@ while (fgets(CS outbuffer, sizeof(outbuffer), f) != NULL)
 nextinput:
     if (*inptr == 0)   /* Refill input buffer */
       {
 nextinput:
     if (*inptr == 0)   /* Refill input buffer */
       {
-      alarm(timeout);
       unsigned char *inbufferp = inbuffer;
       unsigned char *inbufferp = inbuffer;
-      for (;;) {
-      if (srv->tls_active)
-        {
+
+      alarm(timeout);
+      for (;;)
+       {
+       if (srv->tls_active)
+         {
 #ifdef HAVE_OPENSSL
 #ifdef HAVE_OPENSSL
-       int error;
-       DEBUG { printf("call SSL_read\n"); fflush(stdout); }
-        rc = SSL_read(srv->ssl, inbufferp, bsiz - (inbufferp - inbuffer) - 1);
-       DEBUG { printf("SSL_read: %d\n", rc); fflush(stdout); }
-       if (rc <= 0)
-          switch (error = SSL_get_error(srv->ssl, rc))
-           {
-           case SSL_ERROR_ZERO_RETURN:
-             break;
-           case SSL_ERROR_SYSCALL:
-             printf("%s\n", ERR_error_string(ERR_get_error(), NULL));
-             rc = -1;
-             break;
-           case SSL_ERROR_SSL:
-             printf("%s\nTLS terminated\n", ERR_error_string(ERR_get_error(), NULL));
-             SSL_shutdown(srv->ssl);
-             SSL_free(srv->ssl);
-             srv->tls_active = FALSE;
-             { /* OpenSSL leaves it in restartsys mode */
-             struct sigaction act = {.sa_handler = sigalrm_handler_flag, .sa_flags = 0};
-             sigalrm_seen = 1;
-             sigaction(SIGALRM, &act, NULL);
+         int error;
+         DEBUG { printf("call SSL_read\n"); fflush(stdout); }
+         rc = SSL_read(srv->ssl, inbufferp, bsiz - (inbufferp - inbuffer) - 1);
+         DEBUG { printf("SSL_read: %d\n", rc); fflush(stdout); }
+         if (rc <= 0)
+           switch (error = SSL_get_error(srv->ssl, rc))
+             {
+             case SSL_ERROR_ZERO_RETURN:
+               break;
+             case SSL_ERROR_SYSCALL:
+               printf("%s\n", ERR_error_string(ERR_get_error(), NULL));
+               rc = -1;
+               break;
+             case SSL_ERROR_SSL:
+               printf("%s\nTLS terminated\n", ERR_error_string(ERR_get_error(), NULL));
+               SSL_shutdown(srv->ssl);
+               SSL_free(srv->ssl);
+               srv->tls_active = FALSE;
+               {       /* OpenSSL leaves it in restartsys mode */
+               struct sigaction act = {.sa_handler = sigalrm_handler_flag, .sa_flags = 0};
+               sigalrm_seen = 1;
+               sigaction(SIGALRM, &act, NULL);
+               }
+               *inptr = 0;
+               DEBUG { printf("go round\n"); fflush(stdout); }
+               goto nextinput;
+             default:
+               printf("SSL error code %d\n", error);
              }
              }
-             *inptr = 0;
-             DEBUG { printf("go round\n"); fflush(stdout); }
-             goto nextinput;
-           default:
-             printf("SSL error code %d\n", error);
-           }
 #endif
 #ifdef HAVE_GNUTLS
 #endif
 #ifdef HAVE_GNUTLS
-      retry1:
-       DEBUG { printf("call gnutls_record_recv\n"); fflush(stdout); }
-        rc = gnutls_record_recv(tls_session, CS inbufferp, bsiz - (inbufferp - inbuffer) - 1);
-       if (rc < 0)
+       retry1:
+         DEBUG { printf("call gnutls_record_recv\n"); fflush(stdout); }
+         rc = gnutls_record_recv(tls_session, CS inbufferp, bsiz - (inbufferp - inbuffer) - 1);
+         if (rc < 0)
+           {
+           DEBUG { printf("gnutls_record_recv: %s\n", gnutls_strerror(rc)); fflush(stdout); }
+           if (rc == GNUTLS_E_INTERRUPTED || rc == GNUTLS_E_AGAIN)
+             goto retry1;
+           printf("%s\n", gnutls_strerror(rc));
+           srv->tls_active = FALSE;
+           *inptr = 0;
+           DEBUG { printf("go round\n"); fflush(stdout); }
+           goto nextinput;
+           }
+         DEBUG { printf("gnutls_record_recv: %d\n", rc); fflush(stdout); }
+#endif
+         }
+       else
          {
          {
-         DEBUG { printf("gnutls_record_recv: %s\n", gnutls_strerror(rc)); fflush(stdout); }
-         if (rc == GNUTLS_E_INTERRUPTED || rc == GNUTLS_E_AGAIN)
-           goto retry1;
-         printf("%s\n", gnutls_strerror(rc));
-         srv->tls_active = FALSE;
-         *inptr = 0;
-         DEBUG { printf("go round\n"); fflush(stdout); }
-         goto nextinput;
+         DEBUG { printf("call read\n"); fflush(stdout); }
+         rc = read(srv->sock, inbufferp, bsiz - (inbufferp - inbuffer) - 1);
+         DEBUG { printf("read: %d\n", rc); fflush(stdout); }
          }
          }
-       DEBUG { printf("gnutls_record_recv: %d\n", rc); fflush(stdout); }
-#endif
-        }
-      else
-       {
-       DEBUG { printf("call read\n"); fflush(stdout); }
-       rc = read(srv->sock, inbufferp, bsiz - (inbufferp - inbuffer) - 1);
-       DEBUG { printf("read: %d\n", rc); fflush(stdout); }
-       }
 
 
-        if (rc > 0) inbufferp[rc] = '\0';
-        if (rc <= 0 || strchr(inbufferp, '\n')) break;
-        inbufferp += rc;
-        if (inbufferp >= inbuffer + bsiz) {
-          printf("Input buffer overrun, need more than %d bytes input buffer\n", bsiz);
-          exit(73);
-        }
-        DEBUG { printf("read more\n"); }
-      }
+         if (rc > 0) inbufferp[rc] = '\0';
+         if (rc <= 0 || strchr(inbufferp, '\n')) break;
+         inbufferp += rc;
+         if (inbufferp >= inbuffer + bsiz)
+           {
+           printf("Input buffer overrun, need more than %d bytes input buffer\n", bsiz);
+           exit(73);
+           }
+         DEBUG { printf("read more\n"); }
+       }
       alarm(0);
 
       if (rc < 0)
       alarm(0);
 
       if (rc < 0)