diff options
| author | Gregory P. Smith <greg@mad-scientist.com> | 2008-01-19 22:49:37 (GMT) |
|---|---|---|
| committer | Gregory P. Smith <greg@mad-scientist.com> | 2008-01-19 22:49:37 (GMT) |
| commit | 70eb2f91b45e126ca8219d965b1d14968b0405cc (patch) | |
| tree | 72ef3cbac75909f93ab5f195488e1b9b7dd6483a /Lib/subprocess.py | |
| parent | 92ffc634e48268d357f6bde2ed1b7ac26bfb2631 (diff) | |
| download | cpython-70eb2f91b45e126ca8219d965b1d14968b0405cc.zip cpython-70eb2f91b45e126ca8219d965b1d14968b0405cc.tar.gz cpython-70eb2f91b45e126ca8219d965b1d14968b0405cc.tar.bz2 | |
Fix issue 1300: Quote command line arguments that contain a '|' character in
subprocess.list2cmdline (windows).
Diffstat (limited to 'Lib/subprocess.py')
| -rw-r--r-- | Lib/subprocess.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/Lib/subprocess.py b/Lib/subprocess.py index bae4259..975a924 100644 --- a/Lib/subprocess.py +++ b/Lib/subprocess.py @@ -471,8 +471,8 @@ def list2cmdline(seq): 2) A string surrounded by double quotation marks is interpreted as a single argument, regardless of white space - contained within. A quoted string can be embedded in an - argument. + or pipe characters contained within. A quoted string can be + embedded in an argument. 3) A double quotation mark preceded by a backslash is interpreted as a literal double quotation mark. @@ -498,7 +498,7 @@ def list2cmdline(seq): if result: result.append(' ') - needquote = (" " in arg) or ("\t" in arg) or not arg + needquote = (" " in arg) or ("\t" in arg) or ("|" in arg) or not arg if needquote: result.append('"') |