artemist/lix
0
0
Fork 0
forked from lix-project/lix
Code Pull requests Activity

Merge pull request #998 from veprbl/rx_chmod_fix

Browse source 
override rx directory permissions in deletePath()
This commit is contained in:
Eelco Dolstra 2016-10-06 11:08:21 +02:00 committed by GitHub
commit cd128f4bad
1 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/libutil/util.cc
View file

@ -339,10 +339,11 @@ static void _deletePath(const Path & path, unsigned long long & bytesFreed)
bytesFreed += st.st_blocks * 512; bytesFreed += st.st_blocks * 512;
if (S_ISDIR(st.st_mode)) { if (S_ISDIR(st.st_mode)) {
/* Make the directory writable. */ /* Make the directory accessible. */
if (!(st.st_mode & S_IWUSR)) { const auto PERM_MASK = S_IRUSR | S_IWUSR | S_IXUSR;
if (chmod(path.c_str(), st.st_mode | S_IWUSR) == -1) if ((st.st_mode & PERM_MASK) != PERM_MASK) {
throw SysError(format("making ‘%1%’ writable") % path); if (chmod(path.c_str(), st.st_mode | PERM_MASK) == -1)
throw SysError(format("chmod ‘%1%’") % path);
} }
for (auto & i : readDirectory(path)) for (auto & i : readDirectory(path))