commit 348c8aba0d2df2996e0fe57900ef518c6aeb4b29
author Kenny Root <kroot@google.com> Thu May 10 15:39:53 2012 -0700
committer Kenny Root <kroot@google.com> Thu May 10 15:41:16 2012 -0700
tree b134ff8e6bddf13a2d52e692d1429d7df395c32a
parent e5032c42da3c33a854df0a24a7968b4ab54190b9

Only set permissions on dirs or files

Traversal would mark directories with the correct permissions, but
they're visited again in post-order which is a different fts_info flag.
Then it would set that to regular file permissions.

Explicitly check to make sure we're looking at a file instead.

Bug: 6478606
Change-Id: I13cab3e69f451da6a994fa974d575ef366f82025

VolumeManager.cpp

diff --git a/VolumeManager.cpp b/VolumeManager.cpp
index 3a63a19..fe1ce8e 100644
--- a/VolumeManager.cpp
+++ b/VolumeManager.cpp
@@ -571,7 +571,7 @@
 
             if (ftsent->fts_info & FTS_D) {
                 result |= fchmod(fd, 0711);
-            } else {
+            } else if (ftsent->fts_info & FTS_F) {
                 result |= fchmod(fd, privateFile ? 0640 : 0644);
             }
             close(fd);