summaryrefslogtreecommitdiff
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
parent16c1e444f33c2cf6e292458e65e74c263d2d6b9e (diff)
downloadcurl-f5a77473c5397982a9e6ad74de20ccab3d5b31cc.tar.gz
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
-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 {