summaryrefslogtreecommitdiff
path: root/src/tool_formparse.c
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2011-12-16 11:43:25 +0100
committerDaniel Stenberg <daniel@haxx.se>2011-12-16 11:43:25 +0100
commit7b8590d1f5bc332ad49300c698a7992939eb05a9 (patch)
tree121fd4fae3ac573dfb3ed0c3f34bb19c438d0670 /src/tool_formparse.c
parent9b185aac43f0fff9d1523acf41a2318ab91d0d13 (diff)
downloadcurl-7b8590d1f5bc332ad49300c698a7992939eb05a9.tar.gz
curl -F: fix multiple file upload with custom type
Test case 1315 was added to verify this functionality. When passing in multiple files to a single -F, the parser would get all confused if one of the specified files had a custom type= assigned. Reported by: Colin Hogben
Diffstat (limited to 'src/tool_formparse.c')
-rw-r--r--src/tool_formparse.c54
1 files changed, 21 insertions, 33 deletions
diff --git a/src/tool_formparse.c b/src/tool_formparse.c
index 5d6263e5c..ff2210ac8 100644
--- a/src/tool_formparse.c
+++ b/src/tool_formparse.c
@@ -73,9 +73,6 @@
*
***************************************************************************/
-#define FORM_FILE_SEPARATOR ','
-#define FORM_TYPE_SEPARATOR ';'
-
int formparse(struct Configurable *config,
const char *input,
struct curl_httppost **httppost,
@@ -120,8 +117,8 @@ int formparse(struct Configurable *config,
char *ptr;
char *filename = NULL;
- sep = strchr(contp, FORM_TYPE_SEPARATOR);
- sep2 = strchr(contp, FORM_FILE_SEPARATOR);
+ sep = strchr(contp, ';');
+ sep2 = strchr(contp, ',');
/* pick the closest */
if(sep2 && (sep2 < sep)) {
@@ -133,16 +130,13 @@ int formparse(struct Configurable *config,
type = NULL;
if(sep) {
-
- /* if we got here on a comma, don't do much */
- if(FORM_FILE_SEPARATOR == *sep)
- ptr = NULL;
- else
- ptr = sep+1;
+ bool semicolon = (';' == *sep);
*sep = '\0'; /* terminate file name at separator */
- while(ptr && (FORM_FILE_SEPARATOR!= *ptr)) {
+ ptr = sep+1; /* point to the text following the separator */
+
+ while(semicolon && ptr && (','!= *ptr)) {
/* pass all white spaces */
while(ISSPACE(*ptr))
@@ -168,13 +162,17 @@ int formparse(struct Configurable *config,
specified and if not we simply assume that it is text that
the user wants included in the type and include that too up
to the next zero or semicolon. */
- if((*sep==';') && !checkprefix(";filename=", sep)) {
- sep2 = strchr(sep+1, ';');
- if(sep2)
- sep = sep2;
- else
- sep = sep + strlen(sep); /* point to end of string */
+ if(*sep==';') {
+ if(!checkprefix(";filename=", sep)) {
+ sep2 = strchr(sep+1, ';');
+ if(sep2)
+ sep = sep2;
+ else
+ sep = sep + strlen(sep); /* point to end of string */
+ }
}
+ else
+ semicolon = FALSE;
if(*sep) {
*sep = '\0'; /* zero terminate type string */
@@ -186,9 +184,9 @@ int formparse(struct Configurable *config,
}
else if(checkprefix("filename=", ptr)) {
filename = &ptr[9];
- ptr = strchr(filename, FORM_TYPE_SEPARATOR);
+ ptr = strchr(filename, ';');
if(!ptr) {
- ptr = strchr(filename, FORM_FILE_SEPARATOR);
+ ptr = strchr(filename, ',');
}
if(ptr) {
*ptr = '\0'; /* zero terminate */
@@ -199,20 +197,10 @@ int formparse(struct Configurable *config,
/* confusion, bail out of loop */
break;
}
- /* find the following comma */
- if(ptr)
- sep = strchr(ptr, FORM_FILE_SEPARATOR);
- else
- sep = NULL;
- }
- else {
- sep = strchr(contp, FORM_FILE_SEPARATOR);
- }
- if(sep) {
- /* the next file name starts here */
- *sep = '\0';
- sep++;
+
+ sep = ptr;
}
+
/* if type == NULL curl_formadd takes care of the problem */
if(!AddMultiFiles(contp, type, filename, &multi_start,