Commit 6d7bf811 by Michael DeHaan

Merge pull request #3047 from kavink/devel

Avoid crash in module_common when the file cannot be moved/copy
parents dbfad567 8217e070
...@@ -815,7 +815,7 @@ class AnsibleModule(object): ...@@ -815,7 +815,7 @@ class AnsibleModule(object):
self.fail_json(msg='Could not make backup of %s to %s: %s' % (fn, backupdest, e)) self.fail_json(msg='Could not make backup of %s to %s: %s' % (fn, backupdest, e))
return backupdest return backupdest
def cleanup(tmpfile): def cleanup(self,tmpfile):
if os.path.exists(tmpfile): if os.path.exists(tmpfile):
try: try:
os.unlink(tmpfile) os.unlink(tmpfile)
...@@ -856,7 +856,7 @@ class AnsibleModule(object): ...@@ -856,7 +856,7 @@ class AnsibleModule(object):
# rename might not preserve context # rename might not preserve context
self.set_context_if_different(dest, context, False) self.set_context_if_different(dest, context, False)
except (shutil.Error, OSError, IOError), e: except (shutil.Error, OSError, IOError), e:
cleanup(tmp_dest) self.cleanup(tmp_dest)
self.fail_json(msg='Could not replace file: %s to %s: %s' % (src, dest, e)) self.fail_json(msg='Could not replace file: %s to %s: %s' % (src, dest, e))
def run_command(self, args, check_rc=False, close_fds=False, executable=None, data=None): def run_command(self, args, check_rc=False, close_fds=False, executable=None, data=None):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment