diff options
| author | Rob Crittenden <rcritten@redhat.com> | 2011-07-01 10:41:42 -0400 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2011-07-14 19:11:49 -0400 |
| commit | 37e3bf2a6096ea18f46501bf5f2a51c55e829595 (patch) | |
| tree | eaaeefddf55f8db6ba02346afda9eda854b04503 /install/updates/40-delegation.update | |
| parent | 4bd85ceb90d6b8639c14e68691f2c9f447980c2d (diff) | |
| download | freeipa-37e3bf2a6096ea18f46501bf5f2a51c55e829595.tar.gz freeipa-37e3bf2a6096ea18f46501bf5f2a51c55e829595.tar.xz freeipa-37e3bf2a6096ea18f46501bf5f2a51c55e829595.zip | |
Disallow direct modifications to enrolledBy.
This fixes a regression.
We don't need to allow enrolledBy to be modified because it gets
written in the ipa_enrollment plugin which does internal operations
so bypasses acis.
https://fedorahosted.org/freeipa/ticket/302
Diffstat (limited to 'install/updates/40-delegation.update')
| -rw-r--r-- | install/updates/40-delegation.update | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/install/updates/40-delegation.update b/install/updates/40-delegation.update index 96cc59e74..66c62ed54 100644 --- a/install/updates/40-delegation.update +++ b/install/updates/40-delegation.update @@ -258,3 +258,7 @@ add:aci: '(target = "ldap:///fqdn=*,cn=computers,cn=accounts,$SUFFIX")(targetfil dn: cn=Host Enrollment,cn=privileges,cn=pbac,$SUFFIX add:member: 'cn=admins,cn=groups,cn=accounts,$SUFFIX' + +# Don't allow admins to update enrolledBy +dn: $SUFFIX +replace:aci:'(targetattr = "enrolledby || objectclass")(target = "ldap:///fqdn=*,cn=computers,cn=accounts,$SUFFIX")(version 3.0;acl "permission:Enroll a host";allow (write) groupdn = "ldap:///cn=Enroll a host,cn=permissions,cn=pbac,$SUFFIX";)::(targetattr = "objectclass")(target = "ldap:///fqdn=*,cn=computers,cn=accounts,$SUFFIX")(version 3.0;acl "permission:Enroll a host";allow (write) groupdn = "ldap:///cn=Enroll a host,cn=permissions,cn=pbac,$SUFFIX";)' |