almalinux/lorax
12
0
Fork 0
Code Issues Pull Requests Releases Activity

Turn on o+x permission for the queue and results directories

Browse Source 
Otherwise it won't allow a group member to delete things.
This commit is contained in:
Brian C. Lane 2018-02-01 17:31:54 -08:00
parent fdcf839753
commit 3fdf3a9819
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/sbin/lorax-composer
View File

@ -198,13 +198,13 @@ if __name__ == '__main__':
if not os.path.exists(p_dir): if not os.path.exists(p_dir):
log.info("%s does not exist, creating it.", p_dir) log.info("%s does not exist, creating it.", p_dir)
orig_umask = os.umask(0) orig_umask = os.umask(0)
os.makedirs(p_dir, 0o770) os.makedirs(p_dir, 0o771)
os.chown(p_dir, 0, gid) os.chown(p_dir, 0, gid)
os.umask(orig_umask) os.umask(orig_umask)
else: else:
p_stat = os.stat(p_dir) p_stat = os.stat(p_dir)
if p_stat.st_mode & 0o007 != 0: if p_stat.st_mode & 0o006 != 0:
errors.append("Incorrect permissions on %s, no 'other' permissions are allowed." % p_dir) errors.append("Incorrect permissions on %s, no o+rw permissions are allowed." % p_dir)
if p_stat.st_gid != gid or p_stat.st_uid != 0: if p_stat.st_gid != gid or p_stat.st_uid != 0:
errors.append("%s should be owned by root:%s" % (p_dir, opts.group)) errors.append("%s should be owned by root:%s" % (p_dir, opts.group))