Limit mhparam's exit status to 120 missing components.

[nmh] / mts / smtp / smtp.c

diff --git a/mts/smtp/smtp.c b/mts/smtp/smtp.c
index c94ac31308063d2de637c531b857779b5051bae3..0e6e5116199ae6401d77bcb707622d24c763dac5 100644 (file)
--- a/mts/smtp/smtp.c
+++ b/mts/smtp/smtp.c
@@ -40,9 +40,6 @@
  */
 #endif
 
  */
 #endif
 

-#define        TRUE    1
-#define        FALSE   0
-

 #define        NBITS ((sizeof (int)) * 8)
 
 /*
 #define        NBITS ((sizeof (int)) * 8)
 
 /*


@@ -103,9 +100,9 @@ sm_init (char *client, char *server, char *port, int watch, int verbose,
     if (sm_mts == MTS_SMTP)
        return smtp_init (client, server, port, watch, verbose,
                          debug, sasl, saslmech, user, oauth_svc, tls);
     if (sm_mts == MTS_SMTP)
        return smtp_init (client, server, port, watch, verbose,
                          debug, sasl, saslmech, user, oauth_svc, tls);

-    else
-       return sendmail_init (client, watch, verbose, debug, sasl,
-                             saslmech, user);
+
+    return sendmail_init (client, watch, verbose, debug, sasl,
+                          saslmech, user);

 }
 
 static int
 }
 
 static int


@@ -131,26 +128,27 @@ smtp_init (char *client, char *server, char *port, int watch, int verbose,
        } else {
            client = LocalName(1);      /* no clientname -> LocalName */
        }
        } else {
            client = LocalName(1);      /* no clientname -> LocalName */
        }

-    }

 
 

-    /*
-     * Last-ditch check just in case client still isn't set to anything
-     */
-
-    if (client == NULL || *client == '\0')
-       client = "localhost";
+        /*
+         * Last-ditch check just in case client still isn't set to anything
+         */
+        if (client == NULL || *client == '\0')
+            client = "localhost";
+    }

 
     nsc = netsec_init();
 
     if (user)
        netsec_set_userid(nsc, user);
 
 
     nsc = netsec_init();
 
     if (user)
        netsec_set_userid(nsc, user);
 

+    netsec_set_hostname(nsc, server);
+

     if (sm_debug)
        netsec_set_snoop(nsc, 1);
 
     if (sasl) {
     if (sm_debug)
        netsec_set_snoop(nsc, 1);
 
     if (sasl) {

-       if (netsec_set_sasl_params(nsc, server, "smtp", saslmech,
-                                  sm_sasl_callback, &errstr) != OK)
+       if (netsec_set_sasl_params(nsc, "smtp", saslmech, sm_sasl_callback,
+                                  &errstr) != OK)

            return sm_nerror(errstr);
     }
 
            return sm_nerror(errstr);
     }
 


@@ -166,18 +164,18 @@ smtp_init (char *client, char *server, char *port, int watch, int verbose,
 
     netsec_set_fd(nsc, sd1, sd1);
 
 
     netsec_set_fd(nsc, sd1, sd1);
 

-    if (tls) {
-       if (netsec_set_tls(nsc, 1, &errstr) != OK)
+    if (tls & S_TLSENABLEMASK) {
+       if (netsec_set_tls(nsc, 1, tls & S_NOVERIFY, &errstr) != OK)

            return sm_nerror(errstr);
     }
 
     /*
            return sm_nerror(errstr);
     }
 
     /*

-     * If tls == 2, that means that the user requested "initial" TLS,
-     * which happens right after the connection has opened.  Do that
-     * negotiation now
+     * If tls == S_INITTLS, that means that the user requested
+     * "initial" TLS, which happens right after the connection has
+     * opened.  Do that negotiation now

      */
 
      */
 

-    if (tls == 2) {
+    if (tls & S_INITTLS) {

        if (netsec_negotiate_tls(nsc, &errstr) != OK) {
            sm_end(NOTOK);
            return sm_nerror(errstr);
        if (netsec_negotiate_tls(nsc, &errstr) != OK) {
            sm_end(NOTOK);
            return sm_nerror(errstr);


@@ -218,7 +216,7 @@ smtp_init (char *client, char *server, char *port, int watch, int verbose,
      * restart the EHLO dialog after TLS negotiation is complete.
      */
 
      * restart the EHLO dialog after TLS negotiation is complete.
      */
 

-    if (tls == 1) {
+    if (tls & S_STARTTLS) {

        if (! EHLOset("STARTTLS")) {
            sm_end(NOTOK);
            return sm_ierror("SMTP server does not support TLS");
        if (! EHLOset("STARTTLS")) {
            sm_end(NOTOK);
            return sm_ierror("SMTP server does not support TLS");


@@ -299,26 +297,27 @@ sendmail_init (char *client, int watch, int verbose, int debug, int sasl,
            client = clientname;
        else
            client = LocalName(1);      /* no clientname -> LocalName */
            client = clientname;
        else
            client = LocalName(1);      /* no clientname -> LocalName */

-    }

 
 

-    /*
-     * Last-ditch check just in case client still isn't set to anything
-     */
-
-    if (client == NULL || *client == '\0')
-       client = "localhost";
+        /*
+         * Last-ditch check just in case client still isn't set to anything
+         */
+        if (client == NULL || *client == '\0')
+            client = "localhost";
+    }

 
     nsc = netsec_init();
 
     if (user)
        netsec_set_userid(nsc, user);
 
 
     nsc = netsec_init();
 
     if (user)
        netsec_set_userid(nsc, user);
 

+    netsec_set_hostname(nsc, client);
+

     if (sm_debug)
        netsec_set_snoop(nsc, 1);
 
     if (sasl) {
     if (sm_debug)
        netsec_set_snoop(nsc, 1);
 
     if (sasl) {

-       if (netsec_set_sasl_params(nsc, client, "smtp", saslmech,
-                                  sm_sasl_callback, &errstr) != OK)
+       if (netsec_set_sasl_params(nsc, "smtp", saslmech, sm_sasl_callback,
+                                  &errstr) != OK)

            return sm_nerror(errstr);
     }
 
            return sm_nerror(errstr);
     }
 


@@ -453,6 +452,9 @@ sm_winit (char *from, int smtputf8, int eightbit)
         if (EHLOset ("8BITMIME")  &&  EHLOset ("SMTPUTF8")) {
             mail_parameters = " BODY=8BITMIME SMTPUTF8";
         } else {
         if (EHLOset ("8BITMIME")  &&  EHLOset ("SMTPUTF8")) {
             mail_parameters = " BODY=8BITMIME SMTPUTF8";
         } else {

+            advise (NULL, "SMTP server does not support %s, not sending.\n"
+                    "Rebuild message with 7-bit headers, WITHOUT -headerencoding utf-8.",
+                    EHLOset ("SMTPUTF8") ? "8BITMIME" : "SMTPUTF8");

             sm_end (NOTOK);
             return RP_UCMD;
         }
             sm_end (NOTOK);
             return RP_UCMD;
         }


@@ -464,7 +466,7 @@ sm_winit (char *from, int smtputf8, int eightbit)
         } else {
             advise (NULL, "SMTP server does not support 8BITMIME, not sending.\n"
                     "Suggest encoding message for 7-bit transport by setting your\n"
         } else {
             advise (NULL, "SMTP server does not support 8BITMIME, not sending.\n"
                     "Suggest encoding message for 7-bit transport by setting your\n"

-                    "locale to C, and/or specifying *b64 in mhbuild directives.\n");
+                    "locale to C, and/or specifying *b64 in mhbuild directives.");

             sm_end (NOTOK);
             return RP_UCMD;
         }
             sm_end (NOTOK);
             return RP_UCMD;
         }


@@ -813,7 +815,7 @@ again: ;
                                                 &errstr)) != NULL ; ) {
 
        if (doingEHLO
                                                 &errstr)) != NULL ; ) {
 
        if (doingEHLO

-               && strncmp (buffer, "250", sizeof("250") - 1) == 0
+               && has_prefix(buffer, "250")

                && (buffer[3] == '-' || doingEHLO == 2)
                && buffer[4]) {
            if (doingEHLO == 2) {
                && (buffer[3] == '-' || doingEHLO == 2)
                && buffer[4]) {
            if (doingEHLO == 2) {


@@ -875,7 +877,7 @@ again: ;
            continue;
        if (sm_reply.code < 100) {
            if (sm_verbose) {
            continue;
        if (sm_reply.code < 100) {
            if (sm_verbose) {

-               printf ("%s\n", sm_reply.text);
+               puts(sm_reply.text);

                fflush (stdout);
            }
            goto again;
                fflush (stdout);
            }
            goto again;


@@ -952,7 +954,7 @@ EHLOset (char *s)
 
     for (ehlo = EHLOkeys; *ehlo; ehlo++) {
        ep = *ehlo;
 
     for (ehlo = EHLOkeys; *ehlo; ehlo++) {
        ep = *ehlo;

-       if (strncmp (ep, s, len) == 0) {
+       if (has_prefix(ep, s)) {

            for (ep += len; *ep == ' '; ep++)
                continue;
            return ep;
            for (ep += len; *ep == ' '; ep++)
                continue;
            return ep;


@@ -1030,7 +1032,7 @@ sm_sasl_callback(enum sasl_message_type mtype, unsigned const char *indata,
            return NOTOK;
        }
 
            return NOTOK;
        }
 

-       if (strncmp(line, "334 ", 4) != 0) {
+       if (!has_prefix(line, "334 ")) {

            netsec_err(errstr, "Improper SASL protocol response: %s", line);
            return NOTOK;
        }
            netsec_err(errstr, "Improper SASL protocol response: %s", line);
            return NOTOK;
        }


@@ -1079,7 +1081,7 @@ sm_sasl_callback(enum sasl_message_type mtype, unsigned const char *indata,
        if (line == NULL)
            return NOTOK;
 
        if (line == NULL)
            return NOTOK;
 

-       if (strncmp(line, "235 ", 4) != 0) {
+       if (!has_prefix(line, "235 ")) {

            if (len > 4)
                netsec_err(errstr, "Authentication failed: %s", line + 4);
            else
            if (len > 4)
                netsec_err(errstr, "Authentication failed: %s", line + 4);
            else