summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorPatrick Monnerat <patrick@monnerat.net>2017-09-03 16:10:55 +0100
committerPatrick Monnerat <patrick@monnerat.net>2017-09-03 16:10:55 +0100
commitefd93014260e8a7e36e12c1e9c5e4296dd5c5a9e (patch)
tree6d8f375f3d9a15eea59a4362cb966ec360a416a9 /src
parent1a3f4c19919b09ff0fd0ba228b06a217d85d4265 (diff)
downloadcurl-efd93014260e8a7e36e12c1e9c5e4296dd5c5a9e.tar.gz
mime: use size_t instead of ssize_t in public API interface.
To support telling a string is nul-terminated, symbol CURL_ZERO_TERMINATED has been introduced. Documentation updated accordingly. symbols in versions updated. Added form API symbols deprecation info.
Diffstat (limited to 'src')
-rw-r--r--src/tool_setopt.c18
1 files changed, 10 insertions, 8 deletions
diff --git a/src/tool_setopt.c b/src/tool_setopt.c
index 19646ea69..967241d64 100644
--- a/src/tool_setopt.c
+++ b/src/tool_setopt.c
@@ -210,12 +210,14 @@ 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 plen)
+static char *c_escape(const char *str, size_t len)
{
const char *s;
unsigned char c;
char *escaped, *e;
- size_t len = plen == -1? strlen(str): (size_t) plen;
+
+ if(len == CURL_ZERO_TERMINATED)
+ len = strlen(str);
/* Check for possible overflow. */
if(len > (~(size_t) 0) / 4)
@@ -392,7 +394,7 @@ static CURLcode libcurl_generate_slist(struct curl_slist *slist, int *slistno)
CLEAN1("slist%d = NULL;", *slistno);
for(; slist; slist = slist->next) {
Curl_safefree(escaped);
- escaped = c_escape(slist->data, -1);
+ escaped = c_escape(slist->data, CURL_ZERO_TERMINATED);
if(!escaped)
return CURLE_OUT_OF_MEMORY;
DATA3("slist%d = curl_slist_append(slist%d, \"%s\");",
@@ -432,7 +434,7 @@ static CURLcode libcurl_generate_mime(curl_mime *mime, int *mimeno)
switch(part->kind) {
case MIMEKIND_NAMEDFILE:
Curl_safefree(escaped);
- escaped = c_escape(part->data, -1);
+ escaped = c_escape(part->data, CURL_ZERO_TERMINATED);
if(!escaped)
return CURLE_OUT_OF_MEMORY;
CODE2("curl_mime_filedata(part%d, \"%s\");", *mimeno, escaped);
@@ -480,7 +482,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, (ssize_t) part->datasize);
+ escaped = c_escape(data, (size_t) part->datasize);
if(data != part->data)
Curl_safefree(data);
if(!escaped)
@@ -502,7 +504,7 @@ static CURLcode libcurl_generate_mime(curl_mime *mime, int *mimeno)
if(filename) {
Curl_safefree(escaped);
- escaped = c_escape(filename, -1);
+ escaped = c_escape(filename, CURL_ZERO_TERMINATED);
if(!escaped)
return CURLE_OUT_OF_MEMORY;
CODE2("curl_mime_filename(part%d, \"%s\");", *mimeno, escaped);
@@ -523,7 +525,7 @@ static CURLcode libcurl_generate_mime(curl_mime *mime, int *mimeno)
if(part->mimetype) {
Curl_safefree(escaped);
- escaped = c_escape(part->mimetype, -1);
+ escaped = c_escape(part->mimetype, CURL_ZERO_TERMINATED);
if(!escaped)
return CURLE_OUT_OF_MEMORY;
CODE2("curl_mime_type(part%d, \"%s\");", *mimeno, escaped);
@@ -674,7 +676,7 @@ CURLcode tool_setopt(CURL *curl, bool str, struct GlobalConfig *config,
REM2("%s set to a %s", name, value);
else {
if(escape) {
- escaped = c_escape(value, -1);
+ escaped = c_escape(value, CURL_ZERO_TERMINATED);
if(!escaped) {
ret = CURLE_OUT_OF_MEMORY;
goto nomem;