diff options
| author | Markus Juenemann <markus@juenemann.net> | 2015-06-16 10:37:38 +1000 |
|---|---|---|
| committer | Markus Juenemann <markus@juenemann.net> | 2015-06-16 10:37:38 +1000 |
| commit | 0db3206cae970deec940d1e7d6d447ab9a0bbe5b (patch) | |
| tree | c8aeb84d774c200fb68be7400a502699e12636a0 | |
| parent | 3405f69281ddd63fdede4c5f13f06365c769de74 (diff) | |
| download | ansible-modules-core-0db3206cae970deec940d1e7d6d447ab9a0bbe5b.tar.gz | |
Removed check whether destination directory is writable from files/unarchive.py.
This check will prevent extraction of an archive if the archive does not
actually write to the destination directory but only writes to any writable
sub-directories of it. The underlying tar command will report errors
should it try to write to read-only directories.
| -rw-r--r-- | files/unarchive.py | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/files/unarchive.py b/files/unarchive.py index 386503ca..e25d0d75 100644 --- a/files/unarchive.py +++ b/files/unarchive.py @@ -277,8 +277,6 @@ def main(): # is dest OK to receive tar file? if not os.path.isdir(dest): module.fail_json(msg="Destination '%s' is not a directory" % dest) - if not os.access(dest, os.W_OK): - module.fail_json(msg="Destination '%s' not writable" % dest) handler = pick_handler(src, dest, module) |