diff options
author | Csaba Henk <chenk@redhat.com> | 2014-10-19 01:01:01 +0200 |
---|---|---|
committer | Csaba Henk <chenk@redhat.com> | 2015-02-14 02:00:04 +0100 |
commit | c20a86d550c22a8d4bf7b4f413d14d96eccb43ef (patch) | |
tree | ad84948f2379af42161266e351a6b3c975908094 /oslo_concurrency/processutils.py | |
parent | df35680b673eabe0b664f17663f2272711b2e5b4 (diff) | |
download | oslo-concurrency-c20a86d550c22a8d4bf7b4f413d14d96eccb43ef.tar.gz |
processutils: execute(): fix option incompatibility
Issue: simultaneous usage of 'shell' and 'run_as_root' options led
to error.
Cause: it's actually a concealed TypeError: in 'shell'
mode the command argument is assumed to be a string, elsewhere
a list/tuple.
Fix: the command editing implied by 'run_as_root' is performed
with care taken of above tacit type assumption.
Change-Id: Iaba33e6fda67793ae2874ba278c990271ee5e47f
Closes-Bug: #1382873
Diffstat (limited to 'oslo_concurrency/processutils.py')
-rw-r--r-- | oslo_concurrency/processutils.py | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/oslo_concurrency/processutils.py b/oslo_concurrency/processutils.py index 78f8f14..9892aba 100644 --- a/oslo_concurrency/processutils.py +++ b/oslo_concurrency/processutils.py @@ -189,7 +189,12 @@ def execute(*cmd, **kwargs): raise NoRootWrapSpecified( message=_('Command requested root, but did not ' 'specify a root helper.')) - cmd = shlex.split(root_helper) + list(cmd) + if shell: + # root helper has to be injected into the command string + cmd = [' '.join((root_helper, cmd[0]))] + list(cmd[1:]) + else: + # root helper has to be tokenized into argument list + cmd = shlex.split(root_helper) + list(cmd) cmd = [str(c) for c in cmd] sanitized_cmd = strutils.mask_password(' '.join(cmd)) |