From: Ralph Corderoy Date: Sun, 23 Oct 2016 17:56:33 +0000 (+0100) Subject: Use HasPrefix() instead of strncmp(). X-Git-Url: https://diplodocus.org/git/nmh/commitdiff_plain/23cc10281b91bba7d815e5a1506859e6a8bd8201?hp=ef1ba39e8dae81091b6c3e73e72825ef6edea3c6 Use HasPrefix() instead of strncmp(). --- diff --git a/mts/smtp/smtp.c b/mts/smtp/smtp.c index 68228ee3..f1b4917c 100644 --- a/mts/smtp/smtp.c +++ b/mts/smtp/smtp.c @@ -811,7 +811,7 @@ again: ; &errstr)) != NULL ; ) { if (doingEHLO - && strncmp (buffer, "250", LEN("250")) == 0 + && HasPrefix(buffer, "250") && (buffer[3] == '-' || doingEHLO == 2) && buffer[4]) { if (doingEHLO == 2) { @@ -950,7 +950,7 @@ EHLOset (char *s) for (ehlo = EHLOkeys; *ehlo; ehlo++) { ep = *ehlo; - if (strncmp (ep, s, len) == 0) { + if (HasPrefix(ep, s)) { for (ep += len; *ep == ' '; ep++) continue; return ep; @@ -1028,7 +1028,7 @@ sm_sasl_callback(enum sasl_message_type mtype, unsigned const char *indata, return NOTOK; } - if (strncmp(line, "334 ", 4) != 0) { + if (!HasPrefix(line, "334 ")) { netsec_err(errstr, "Improper SASL protocol response: %s", line); return NOTOK; } @@ -1077,7 +1077,7 @@ sm_sasl_callback(enum sasl_message_type mtype, unsigned const char *indata, if (line == NULL) return NOTOK; - if (strncmp(line, "235 ", 4) != 0) { + if (!HasPrefix(line, "235 ")) { if (len > 4) netsec_err(errstr, "Authentication failed: %s", line + 4); else