From 352f8339eace92b394a1bebcdab4b424911de459 Mon Sep 17 00:00:00 2001 From: Stephen Fromm Date: Thu, 9 Aug 2012 22:59:39 -0700 Subject: [PATCH] Update file module to not recurse when setting ownership This updates set_owner_if_different() and set_group_if_different() to not implicitly recurse when setting ownership (whether user or group). It drops the os.system() call and replaces it with os.chown(). Resolves issue #825. The recursion should be explicit. A recurse=yes|no option should be added to the file module. --- file | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/file b/file index 2ae5c6d27d2..5692bf477f9 100755 --- a/file +++ b/file @@ -161,8 +161,13 @@ def set_owner_if_different(path, owner, changed): return changed user, group = user_and_group(path) if owner != user: - rc = os.system("/bin/chown -R %s %s 2>/dev/null" % (owner, path)) - if rc != 0: + try: + uid = pwd.getpwnam(owner).pw_uid + except KeyError: + module_fail_json(path=path, msg='chown failed: failed to look up user %s' % owner) + try: + os.chown(path, uid, -1) + except OSError: module_fail_json(path=path, msg='chown failed') return True @@ -173,8 +178,13 @@ def set_group_if_different(path, group, changed): return changed old_user, old_group = user_and_group(path) if old_group != group: - rc = os.system("/bin/chgrp -R %s %s" % (group, path)) - if rc != 0: + try: + gid = grp.getgrnam(group).gr_gid + except KeyError: + module_fail_json(path=path, msg='chgrp failed: failed to look up group %s' % group) + try: + os.chown(path, -1, gid) + except OSError: module_fail_json(path=path, msg='chgrp failed') return True return changed