diff options
author | Jonas Bjurel <jonas.bjurel@ericsson.com> | 2016-09-12 13:29:26 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@172.30.200.206> | 2016-09-12 13:29:26 +0000 |
commit | 0f5a1854028e33fa77127ffc65077058fa3b9469 (patch) | |
tree | 9d06acfce18ffed4d22c3f4826c82a695e53cd03 /deploy/common.py | |
parent | 23a23192e14895fd235e29c63e00e09bafd254de (diff) | |
parent | bddbf2a740b46d1ad763967b8c8662b1877a4d75 (diff) |
Merge "Properly handle quotes in argument masking"
Diffstat (limited to 'deploy/common.py')
-rw-r--r-- | deploy/common.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/deploy/common.py b/deploy/common.py index 353045867..80832e201 100644 --- a/deploy/common.py +++ b/deploy/common.py @@ -18,6 +18,7 @@ import shutil import stat import errno import time +import shlex N = {'id': 0, 'status': 1, 'name': 2, 'cluster': 3, 'ip': 4, 'mac': 5, 'roles': 6, 'pending_roles': 7, 'online': 8, 'group_id': 9} @@ -41,7 +42,7 @@ os.chmod(LOGFILE, 0664) def mask_arguments(cmd, mask_args, mask_str): - cmd_line = cmd.split() + cmd_line = shlex.split(cmd) for pos in mask_args: # Don't mask the actual command; also check if we don't reference # beyond bounds |