From 8a2d91492cbd2224ff9cdb28163836cb34df2da1 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Sat, 11 May 2013 19:21:06 -0700 Subject: maint: add FIXME comment --- src/copy.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) 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) { -- cgit v1.2.3-54-g00ecf