[Freeipa-devel] [PATCHES] 0572-0575 Add ACI.txt + default bind rule type

Petr Viktorin pviktori at redhat.com
Mon Jun 9 18:08:27 UTC 2014


Having another verification tool should help reviewing the permission 
patches.


To avoid conflicts, apply on top of my patches 0568-0570 (Write User 
permissions).


0572: I tried to make the ACIs generated by the permission plugin as 
predictable as possible, but I missed one place it's affected by 
dict/set iteration order (which is undefined). Here's a fix.

0573: Minor refactoring to make the next patch easier.

0574: Add ACI.txt & makeaci. Due to the predictable ACIs, all this needs 
to do is generate the file; comparing can be done bit-by-bit.
I do run the validation results through difflib, but frankly it's easier 
just to use Git.

0575: Make 'permission' the default bind rule type for managed 
permissions. Rationale in the commit message.
Run makeaci to verify this doesn't change the result.


-- 
Petr³
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pviktori-0572-permission-plugin-Sort-rights-when-writing-the-ACI.patch
Type: text/x-patch
Size: 973 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140609/3143d88f/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pviktori-0573-Add-method-to-enumerate-managed-permission-templates.patch
Type: text/x-patch
Size: 3373 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140609/3143d88f/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pviktori-0574-Add-ACI.txt.patch
Type: text/x-patch
Size: 26104 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140609/3143d88f/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pviktori-0575-Make-permission-the-default-bind-type-for-managed-pe.patch
Type: text/x-patch
Size: 12580 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140609/3143d88f/attachment-0003.bin>


More information about the Freeipa-devel mailing list