summaryrefslogtreecommitdiff
path: root/src/tool_setopt.c
diff options
context:
space:
mode:
authorPatrick Monnerat <patrick@monnerat.net>2017-09-03 10:18:58 +0100
committerPatrick Monnerat <patrick@monnerat.net>2017-09-03 10:18:58 +0100
commit045b076ae8362006f6977a80e6a5e3ef0eb903eb (patch)
tree7533b5f52cbd99ccac579e6ba77e669aaab8369c /src/tool_setopt.c
parent3baf36edf6ed1d7c74d68846d79f87334b573aea (diff)
downloadcurl-045b076ae8362006f6977a80e6a5e3ef0eb903eb.tar.gz
mime: fix some implicit curl_off_t --> size_t conversion warnings.
Diffstat (limited to 'src/tool_setopt.c')
-rw-r--r--src/tool_setopt.c12
1 files changed, 8 insertions, 4 deletions
diff --git a/src/tool_setopt.c b/src/tool_setopt.c
index 6cb6bb3e3..4e25e9e12 100644
--- a/src/tool_setopt.c
+++ b/src/tool_setopt.c
@@ -210,14 +210,18 @@ static const NameValue setopt_nv_CURLNONZERODEFAULTS[] = {
/* Escape string to C string syntax. Return NULL if out of memory.
* Is this correct for those wacky EBCDIC guys? */
-static char *c_escape(const char *str, ssize_t len)
+static char *c_escape(const char *str, ssize_t plen)
{
const char *s;
unsigned char c;
char *escaped, *e;
+ size_t len = plen == -1? strlen(str): (size_t) plen;
+
+ /* Check for possible overflow. */
+ if(len > (~(size_t) 0) / 4)
+ return NULL;
+
/* Allocate space based on worst-case */
- if(len < 0)
- len = strlen(str);
escaped = malloc(4 * len + 1);
if(!escaped)
return NULL;
@@ -474,7 +478,7 @@ static CURLcode libcurl_generate_mime(curl_mime *mime, int *mimeno)
;
size = (cp == data + part->datasize)? (curl_off_t) -1: part->datasize;
Curl_safefree(escaped);
- escaped = c_escape(data, part->datasize);
+ escaped = c_escape(data, (ssize_t) part->datasize);
if(data != part->data)
Curl_safefree(data);
if(!escaped)