diff -ur curl-7.13.2-20050310/docs/MANUAL curl-7.13.2-20050310-patched/docs/MANUAL --- curl-7.13.2-20050310/docs/MANUAL Sat Dec 11 14:00:05 2004 +++ curl-7.13.2-20050310-patched/docs/MANUAL Sat Mar 12 12:54:07 2005 @@ -299,6 +299,13 @@ curl -F "docpicture=@dog.gif" -F "catpicture=@cat.gif" + To send a field value literally without interpreting a leading '@' + or '<', or an embedded ';type=', use --form-string instead of + -F. This is recommended when the value is obtained from a user or + some other unpredictable source. Under these circumstances, using + -F instead of --form-string would allow a user to trick curl into + uploading a file. + REFERRER A HTTP request has the option to include information about which address diff -ur curl-7.13.2-20050310/docs/curl.1 curl-7.13.2-20050310-patched/docs/curl.1 --- curl-7.13.2-20050310/docs/curl.1 Fri Feb 18 14:00:06 2005 +++ curl-7.13.2-20050310-patched/docs/curl.1 Sat Mar 12 10:45:10 2005 @@ -388,6 +388,12 @@ See further examples and details in the MANUAL. This option can be used multiple times. +.IP "--form-string " +(HTTP) Similar to \fI--form\fP except that the value string for the named +parameter is used literally. Leading \&'@' and \&'<' characters, and the +\&';type=' string in the value have no special meaning. Use this in +preference to \fI--form\fP if there's any possibility that the string value +may accidentally trigger the \&'@' or \&'<' features of \fI--form\f{. .IP "-g/--globoff" This option switches off the "URL globbing parser". When you set this option, you can specify URLs that contain the letters {}[] without having them being diff -ur curl-7.13.2-20050310/src/main.c curl-7.13.2-20050310-patched/src/main.c --- curl-7.13.2-20050310/src/main.c Tue Mar 8 14:00:06 2005 +++ curl-7.13.2-20050310-patched/src/main.c Sat Mar 12 13:09:04 2005 @@ -355,6 +355,7 @@ " --ftp-pasv Use PASV instead of PORT (F)", " --ftp-ssl Enable SSL/TLS for the ftp transfer (F)", " -F/--form Specify HTTP multipart POST data (H)", + " --form-string Specify HTTP multipart POST data (H)", " -g/--globoff Disable URL sequences and ranges using {} and []", " -G/--get Send the -d data with a HTTP GET (H)", " -h/--help This help text", @@ -774,6 +775,9 @@ * Specify files to upload with 'name=@filename'. Supports specified * given Content-Type of the files. Such as ';type='. * + * If literal_value is set, any initial '@' or '<' in the value string + * loses its special meaning, as does any embedded ';type='. + * * You may specify more than one file for a single name (field). Specify * multiple files by writing it like: * @@ -804,7 +808,8 @@ static int formparse(char *input, struct curl_httppost **httppost, - struct curl_httppost **last_post) + struct curl_httppost **last_post, + bool literal_value) { /* nextarg MUST be a string in the format 'name=contents' and we'll build a linked list with the info */ @@ -829,7 +834,7 @@ } contp = contents; - if('@' == contp[0]) { + if('@' == contp[0] && !literal_value) { struct multi_files *multi_start = NULL, *multi_current = NULL; /* we use the @-letter to indicate file name(s) */ contp++; @@ -974,7 +979,7 @@ else { struct curl_forms info[4]; int i = 0; - char *ct = strstr(contp, ";type="); + char *ct = literal_value? NULL: strstr(contp, ";type="); info[i].option = CURLFORM_COPYNAME; info[i].value = name; @@ -987,7 +992,7 @@ ct[0]=0; /* zero terminate here */ } - if( contp[0]=='<' ) { + if( contp[0]=='<' && !literal_value) { info[i].option = CURLFORM_FILECONTENT; info[i].value = contp+1; i++; @@ -1280,6 +1285,7 @@ {"Eg","capath ", TRUE}, {"f", "fail", FALSE}, {"F", "form", TRUE}, + {"Fs","form-string", TRUE}, {"g", "globoff", FALSE}, {"G", "get", FALSE}, {"h", "help", FALSE}, @@ -1361,8 +1367,10 @@ do { /* we can loop here if we have multiple single-letters */ - if(!longopt) + if(!longopt) { letter = parse?(char)*parse:'\0'; + subletter='\0'; + } else { letter = parse[0]; subletter = parse[1]; @@ -1833,7 +1841,8 @@ to sort this out slowly and carefully */ if(formparse(nextarg, &config->httppost, - &config->last_post)) + &config->last_post, + subletter=='s')) /* 's' means literal string */ return PARAM_BAD_USE; if(SetHTTPrequest(HTTPREQ_POST, &config->httpreq)) return PARAM_BAD_USE; diff -ur curl-7.13.2-20050310/tests/data/test39 curl-7.13.2-20050310-patched/tests/data/test39 --- curl-7.13.2-20050310/tests/data/test39 Wed Jan 26 14:00:08 2005 +++ curl-7.13.2-20050310-patched/tests/data/test39 Sat Mar 12 10:33:05 2005 @@ -19,7 +19,7 @@ HTTP RFC1867-type formposting with filename= and type= -http://%HOSTIP:%HTTPPORT/we/want/39 -F name=daniel -F tool=curl -F "file=@log/test39.txt;filename=fakerfile;type=moo/foobar" -F file2=@log/test39.txt +http://%HOSTIP:%HTTPPORT/we/want/39 -F name=daniel -F tool=curl --form-string "str1=@literal" --form-string "str2= # We create this file before the command is invoked! @@ -41,7 +41,7 @@ Host: 127.0.0.1:%HTTPPORT Pragma: no-cache Accept: */* -Content-Length: 594 +Content-Length: 810 Expect: 100-continue Content-Type: multipart/form-data; boundary=----------------------------24e78000bd32 @@ -53,6 +53,14 @@ Content-Disposition: form-data; name="tool" curl +------------------------------24e78000bd32 +Content-Disposition: form-data; name="str1" + +@literal +------------------------------24e78000bd32 +Content-Disposition: form-data; name="str2" + +