diff options
| author | Paul Eggert <eggert@cs.ucla.edu> | 2013-05-11 19:21:06 -0700 |
|---|---|---|
| committer | Paul Eggert <eggert@cs.ucla.edu> | 2013-05-11 19:22:54 -0700 |
| commit | 8a2d91492cbd2224ff9cdb28163836cb34df2da1 (patch) | |
| tree | 48a1bda4b1965091a6226d2d3aa799377f2e9a71 | |
| parent | e7198a67b8dd7f23925dfa40b7a90342e185f729 (diff) | |
| download | coreutils-8a2d91492cbd2224ff9cdb28163836cb34df2da1.tar.xz | |
maint: add FIXME comment
| -rw-r--r-- | src/copy.c | 6 |
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) { |