From 7508c23f4d13113c93d438ede8326327de93457c Mon Sep 17 00:00:00 2001
From: Sinisa Veseli <sveseli@aps.anl.gov>
Date: Sat, 6 Feb 2016 03:15:29 +0000
Subject: [PATCH] need to put quotes around path in case of special characters
---
src/python/dm/common/utility/ldapLinuxPlatformUtility.py | 4 ++--
src/python/dm/common/utility/linuxUtility.py | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/python/dm/common/utility/ldapLinuxPlatformUtility.py b/src/python/dm/common/utility/ldapLinuxPlatformUtility.py
index ec2581c3..58f32b1a 100755
--- a/src/python/dm/common/utility/ldapLinuxPlatformUtility.py
+++ b/src/python/dm/common/utility/ldapLinuxPlatformUtility.py
@@ -293,14 +293,14 @@ class LdapLinuxPlatformUtility:
""" Set path permissions for the given group. """
logger = cls.getLogger()
logger.debug('Allowing group %s to read/execute path %s' % (groupName, path))
- cmd = '%s -m group\:%s\:rx %s' % (cls.SETFACL_CMD, groupName, path)
+ cmd = '%s -m group\:%s\:rx "%s"' % (cls.SETFACL_CMD, groupName, path)
cls.executeSudoCommand(cmd)
@classmethod
def changePathGroupOwner(cls, path, groupName):
logger = cls.getLogger()
logger.debug('Changing group owner to %s for path %s' % (groupName, path))
- cmd = '%s \:%s %s' % (cls.CHOWN_CMD, groupName, path)
+ cmd = '%s \:%s "%s"' % (cls.CHOWN_CMD, groupName, path)
cls.executeSudoCommand(cmd)
@classmethod
diff --git a/src/python/dm/common/utility/linuxUtility.py b/src/python/dm/common/utility/linuxUtility.py
index 7bfa1835..3517eea3 100755
--- a/src/python/dm/common/utility/linuxUtility.py
+++ b/src/python/dm/common/utility/linuxUtility.py
@@ -75,14 +75,14 @@ class LinuxUtility:
""" Set path permissions for the given group. """
logger = cls.getLogger()
logger.debug('Allowing group %s to read/execute path %s' % (groupName, path))
- cmd = '%s -m group\:%s\:rx %s' % (cls.SETFACL_CMD, groupName, path)
+ cmd = '%s -m group\:%s\:rx "%s"' % (cls.SETFACL_CMD, groupName, path)
cls.executeSudoCommand(cmd)
@classmethod
def changePathGroupOwner(cls, path, groupName):
logger = cls.getLogger()
logger.debug('Changing group owner to %s for path %s' % (groupName, path))
- cmd = '%s \:%s %s' % (cls.CHOWN_CMD, groupName, path)
+ cmd = '%s \:%s "%s"' % (cls.CHOWN_CMD, groupName, path)
cls.executeSudoCommand(cmd)
#######################################################################
--
GitLab