Log In - Home Page

Patch: Default ACLs

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2009-09
Topic Security
Patch Status Committed
Author Petr Jelinek
Reviewers Jan Urbański
Committer Nobody
Close Date 2009-10-05
Comments
Patch by PJMODOS on 2009-09-14 01:47:01 PM: Initial version.
Comment by wulczer on 2009-09-18 06:05:09 AM: patch is missing a header file
Patch by sfrost on 2009-09-18 07:33:10 PM: Updated patch posted here.
Review by wulczer on 2009-09-21 05:10:19 AM: First review
Patch by PJMODOS on 2009-09-22 05:13:11 AM: Patch with fixes per review.
Patch by PJMODOS on 2009-09-23 07:22:01 AM: Few cosmetic changes.
Review by wulczer on 2009-09-24 03:43:18 AM: Second review
Patch by PJMODOS on 2009-09-24 05:37:29 AM: Fixed another dependency bug spotted by reviewer.
Review by wulczer on 2009-09-24 06:11:48 AM: Dependencies still not ideal
Patch by PJMODOS on 2009-09-24 07:25:06 AM: Hopefully fixed by assigning target role as owner of the defaultacls.
Review by wulczer on 2009-09-24 08:36:21 AM: Last issue addressed, final suggestions in mail, but left for the committer to decide.
Review by tgl on 2009-09-28 11:45:11 AM: Still needs a nontrivial amount of work.
Patch by PJMODOS on 2009-09-30 01:25:25 PM: An attempt to fix problems pointed out by Tom.
Patch by PJMODOS on 2009-10-02 11:18:17 AM: Doc fix.

Add Comment

Please log in to comment on this patch.