summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorPaul Eggert <eggert@cs.ucla.edu>2013-05-11 19:21:06 -0700
committerPaul Eggert <eggert@cs.ucla.edu>2013-05-11 19:22:54 -0700
commit8a2d91492cbd2224ff9cdb28163836cb34df2da1 (patch)
tree48a1bda4b1965091a6226d2d3aa799377f2e9a71 /src
parente7198a67b8dd7f23925dfa40b7a90342e185f729 (diff)
downloadcoreutils-8a2d91492cbd2224ff9cdb28163836cb34df2da1.tar.xz
maint: add FIXME comment
Diffstat (limited to 'src')
-rw-r--r--src/copy.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/src/copy.c b/src/copy.c
index 5c0ee1e70..c1c82730d 100644
--- a/src/copy.c
+++ b/src/copy.c
@@ -2747,8 +2747,12 @@ owner_failure_ok (struct cp_options const *x)
return ((errno == EPERM || errno == EINVAL) && !x->owner_privileges);
}
-/* Return the user's umask, caching the result. */
+/* Return the user's umask, caching the result.
+ FIXME: If the destination's parent directory has has a default ACL,
+ some operating systems (e.g., GNU/Linux's "POSIX" ACLs) use that
+ ACL's mask rather than the process umask. Currently, the callers
+ of cached_umask incorrectly assume that this situation cannot occur. */
extern mode_t
cached_umask (void)
{