aboutsummaryrefslogtreecommitdiff
path: root/lib/smtp.c
diff options
context:
space:
mode:
authorRikard Falkeborn <rikard.falkeborn@gmail.com>2019-04-04 23:13:56 +0200
committerJay Satiro <raysatiro@yahoo.com>2019-04-04 21:11:42 -0400
commitf5a77473c5397982a9e6ad74de20ccab3d5b31cc (patch)
tree3617ffbf46c0d19d110a7fe19e39506ec0928ff3 /lib/smtp.c
parent16c1e444f33c2cf6e292458e65e74c263d2d6b9e (diff)
smtp: fix compiler warning
- Fix clang string-plus-int warning. Clang 8 warns about adding a string to an int does not append to the string. Indeed it doesn't, but that was not the intention either. Use array indexing as suggested to silence the warning. There should be no functional changes. (In other words clang warns about "foo"+2 but not &"foo"[2] so use the latter.) smtp.c:1221:29: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int] eob = strdup(SMTP_EOB + 2); ~~~~~~~~~~~~~~~~^~~~ Closes https://github.com/curl/curl/pull/3729
Diffstat (limited to 'lib/smtp.c')
-rw-r--r--lib/smtp.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/smtp.c b/lib/smtp.c
index f3db714b5..fec0026a2 100644
--- a/lib/smtp.c
+++ b/lib/smtp.c
@@ -1218,7 +1218,7 @@ static CURLcode smtp_done(struct connectdata *conn, CURLcode status,
returned CURLE_AGAIN, we duplicate the EOB now rather than when the
bytes written doesn't equal len. */
if(smtp->trailing_crlf || !conn->data->state.infilesize) {
- eob = strdup(SMTP_EOB + 2);
+ eob = strdup(&SMTP_EOB[2]);
len = SMTP_EOB_LEN - 2;
}
else {