Bug introduced in the re-write from 3.6.x to 4.0.x (and beyond).
add_current_ace_to_acl() is checking a pointer, instead of its contents. The variable name used to be a bool value, and the code wasn't changed in one place when it was moved to being a pointer to bool value instead of a bool value.
Fix to follow.
Created attachment 9234 [details]
git-am fix proposed to master
Will update with the patch from master and cherry-pick info once it's been integrated.
Created attachment 9235 [details]
git-am fix that went into master
This is the fix that went into master.
Can be picked for 4.1.0 and 4.0.x by:
git cherry-pick -x e24fcf0f3e1b5b19d97a13786b09f069393b06d8
Required for both 4.0.x and 4.1.0.
Re-assigning to Karolin for inclusion as this is the fix that went into master that applies cleanly to 4.1.0 and 4.0.next.
Already reviewed by Jeremy Allison and Michael Adam.
Pushed to autobuild-v4-1-test and autobuild-v4-0-test.
Pushed to v4-1-test and v4-0-test.
Closing out bug report.