Commit 2b6dadc5 authored by Jay Satiro's avatar Jay Satiro
Browse files

tool_operate: Don't sanitize --output path (Windows) 

Due to path separators being incorrectly sanitized in --output
pathnames, eg -o c:\foo => c__foo

This is a partial revert of 3017d8a8 until I write a proper fix. The
remote-name will continue to be sanitized, but if the user specified an
--output with string replacement (#1, #2, etc) that data is unsanitized
until I finish a fix.

Bug: https://github.com/bagder/curl/issues/624
Reported-by: Octavio Schroeder
parent 96596334

src/tool_operate.c

+0 −11
Original line number Diff line number Diff line
@@ -563,17 +563,6 @@ static CURLcode operate_do(struct GlobalConfig *global, 
            char *storefile = outfile;

            result = glob_match_url(&outfile, storefile, urls);

            Curl_safefree(storefile);



#if defined(MSDOS) || defined(WIN32)

            if(!result) {

              result = sanitize_file_name(&outfile);

              if(result) {

                Curl_safefree(outfile);

                goto show_error;

              }

            }

#endif /* MSDOS || WIN32 */



            if(result) {

              /* bad globbing */

              warnf(config->global, "bad output glob!\n");