So this is correct here. The caller has specified whether or not the ace should apply to children. In the past, we set this on files and directories, and setting it on files had no effect. Now we only set on directories -- this should cause no regression in behavior and makes more sense overall. Tests should be updated to reflect this if they were expecting the old behavior.