Created attachment 10655 [details]
Patch to resolve bug
smbcacls does not read hex permissions strings in set/modify options correctly, as the code is expecting an unsigned int. This results in the permissions being set to 0x0.
Example of broken behaviour:
zoe@server:~$ smbcacls --add "ACL:Domain\AUser:ALLOWED/OI|CI/0x001301ff" //server/share filename.txt
zoe@server:~$ smbcacls //server/share filename.txt
Verified as broken on 4.1.12, 4.1.16. Patch attached.
Perfect fix ! I just came across the same problem and created the same fix :-).
I'll get this into 4.2.next, 4.1.next.
Created attachment 11081 [details]
git-am fix for 4.2.next.
Jim please review for 4.2.next.
Created attachment 11082 [details]
git-am fix for 4.1.next.
Back-port for 4.1.next. Jim please review.
Re-assigning to Kaolin for inclusion in 4.2.next and 4.1.next.
(In reply to Jeremy Allison from comment #4)
Pushed to autobuild-v4-[1|2]-test.
(In reply to Karolin Seeger from comment #5)
Pushed to v4-[1|2]-test.
Closing out bug report.