vstart_runner: upgrade the check for commands to be run as another user

Rectify the condition that checks if command to be issued as another
user using sudo is issued as a single argument after "-c".

Signed-off-by: Rishabh Dave <ridave@redhat.com>
This commit is contained in:
Rishabh Dave 2019-06-20 21:35:38 +05:30
parent da441cb381
commit 21a1b30a3e

View File

@ -264,8 +264,8 @@ class LocalRemote(object):
# the desired effect.
errmsg = 'The entire command to executed as other user should be a ' +\
'single argument.\nargs - %s' % (args)
if ('sudo' in args or 'python' in args or 'python2' in args or
'python3' in args) and '-c' in args:
if 'sudo' in args and '-u' in args and '-c' in args and \
args.count('-c') == 1:
if args.index('-c') != len(args) - 2 and \
args[args.index('-c') + 2].find('-') == -1:
raise RuntimeError(errmsg)