HomePhorge

(#11563) Return INHERIT_ONLY_ACEs
38e8681dfd50Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

(#11563) Return INHERIT_ONLY_ACEs

Previously, when reading a DACL, puppet excluded INHERIT_ONLY_ACEs which are
ACEs set on a directory that only serve to apply permissions to child
containers (dir) and objects (file). In other words, the ACE didn't affect the
permissions of the directory, and wasn't needed to determine the file mode.

However, now we need to be able to roundtrip a security descriptor, e.g. to
only change the owner, but leave other ACEs intact.

This commit modifies puppet to include inherit only aces when reading the
DACL, but continue to exclude them when determining the file mode.

Details

Provenance
Josh Cooper <josh@puppetlabs.com>Authored on
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPU67a5aa6192fc: (#11563) Preserve deny ACEs
Branches
Unknown
Tags
Unknown

Event Timeline

Josh Cooper <josh@puppetlabs.com> committed rPU38e8681dfd50: (#11563) Return INHERIT_ONLY_ACEs (authored by Josh Cooper <josh@puppetlabs.com>).Nov 26 2013, 6:32 PM