ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

Properly parse escaped special arguments

Browse source 
Fixes issue #1134
This commit is contained in:
Daniel Hokka Zakrisson 2012-09-29 21:04:05 +02:00
parent 9377c3f525
commit 9261d3962b
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
library/command
View file

@ -23,6 +23,7 @@ import sys
import datetime import datetime
import traceback import traceback
import shlex import shlex
import pipes
import os import os
DOCUMENTATION = ''' DOCUMENTATION = '''
@ -139,6 +140,7 @@ class CommandModule(AnsibleModule):
params['shell'] = True params['shell'] = True
check_args = shlex.split(args) check_args = shlex.split(args)
l_args = []
for x in check_args: for x in check_args:
if x.startswith("creates="): if x.startswith("creates="):
# do not run the command if the line contains creates=filename # do not run the command if the line contains creates=filename
@ -154,7 +156,6 @@ class CommandModule(AnsibleModule):
stderr=False, stderr=False,
rc=0 rc=0
) )
args = args.replace(x,'')
elif x.startswith("removes="): elif x.startswith("removes="):
# do not run the command if the line contains removes=filename # do not run the command if the line contains removes=filename
# and the filename do not exists. This allows idempotence # and the filename do not exists. This allows idempotence
@ -169,7 +170,6 @@ class CommandModule(AnsibleModule):
stderr=False, stderr=False,
rc=0 rc=0
) )
args = args.replace(x,'')
elif x.startswith("chdir="): elif x.startswith("chdir="):
(k,v) = x.split("=", 1) (k,v) = x.split("=", 1)
v = os.path.expanduser(v) v = os.path.expanduser(v)
@ -178,8 +178,9 @@ class CommandModule(AnsibleModule):
elif v[0] != '/': elif v[0] != '/':
self.fail_json(msg="the path for 'chdir' argument must be fully qualified") self.fail_json(msg="the path for 'chdir' argument must be fully qualified")
params['chdir'] = v params['chdir'] = v
args = args.replace(x, '') else:
params['args'] = args l_args.append(x)
return (params, args) params['args'] = " ".join([pipes.quote(x) for x in l_args])
return (params, params['args'])
main() main()