From 045b076ae8362006f6977a80e6a5e3ef0eb903eb Mon Sep 17 00:00:00 2001 From: Patrick Monnerat Date: Sun, 3 Sep 2017 10:18:58 +0100 Subject: mime: fix some implicit curl_off_t --> size_t conversion warnings. --- src/tool_setopt.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) (limited to 'src/tool_setopt.c') 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) -- cgit v1.2.3