CLOUDSTACK-710: CitrixResourceBase, if dest dir is not found, mkdir -m 700 -p
This is a security failsafe, so even if destination does not exist we mkdir the path with 0700 permission. If path exists mkdir -m 700 -p won't do anything. Signed-off-by: Rohit Yadav <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/3a0c99b0 Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/3a0c99b0 Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/3a0c99b0 Branch: refs/heads/ui-plugins Commit: 3a0c99b0a430b12a492f2b93dec8d110603b43b4 Parents: 2e2ee2f Author: Rohit Yadav <[email protected]> Authored: Wed Feb 13 22:45:32 2013 +0530 Committer: Rohit Yadav <[email protected]> Committed: Wed Feb 13 22:45:32 2013 +0530 ---------------------------------------------------------------------- .../xen/resource/CitrixResourceBase.java | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/3a0c99b0/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java ---------------------------------------------------------------------- diff --git a/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java b/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java index c7ff5c7..b3cb54f 100644 --- a/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java +++ b/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java @@ -4763,7 +4763,7 @@ public abstract class CitrixResourceBase implements ServerResource, HypervisorRe s_logger.debug("Copying " + f + " to " + d + " on " + hr.address + " with permission " + p); } try { - session.execCommand("mkdir -p " + d); + session.execCommand("mkdir -m 700 -p " + d); } catch (IOException e) { s_logger.debug("Unable to create destination path: " + d + " on " + hr.address + " but trying anyway");
