diff options
| author | Simo Sorce <ssorce@redhat.com> | 2008-08-15 17:17:03 -0400 |
|---|---|---|
| committer | Simo Sorce <ssorce@redhat.com> | 2008-09-10 15:07:33 -0400 |
| commit | 8e7c98eb7ffc58be01e323af74c8df6d49208405 (patch) | |
| tree | 6eb2806db54fb676b1ae5d845347ad8d6f0da63c /ipa-server/ipaserver | |
| parent | 89ed5a02770c54369bb6b84a33c440f1165fec32 (diff) | |
| download | freeipa-8e7c98eb7ffc58be01e323af74c8df6d49208405.tar.gz freeipa-8e7c98eb7ffc58be01e323af74c8df6d49208405.tar.xz freeipa-8e7c98eb7ffc58be01e323af74c8df6d49208405.zip | |
CVE 2008 3274 related fixes
Diffstat (limited to 'ipa-server/ipaserver')
| -rw-r--r-- | ipa-server/ipaserver/krbinstance.py | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/ipa-server/ipaserver/krbinstance.py b/ipa-server/ipaserver/krbinstance.py index 4cdb801ee..598c2b200 100644 --- a/ipa-server/ipaserver/krbinstance.py +++ b/ipa-server/ipaserver/krbinstance.py @@ -339,6 +339,10 @@ class KrbInstance(service.Service): def __add_pwd_extop_module(self): self.__ldap_mod("pwd-extop-conf.ldif") +KRBMKEY_DENY_ACI = """ +(targetattr = "krbMKey")(version 3.0; acl "No external access"; deny (all) userdn != "ldap:///uid=kdc,cn=sysaccounts,cn=etc,$SUFFIX";) +""" + def __add_master_key(self): #get the Master Key from the stash file try: @@ -358,7 +362,9 @@ class KrbInstance(service.Service): asn1key = pyasn1.codec.ber.encoder.encode(krbMKey) dn = "cn="+self.realm+",cn=kerberos,"+self.suffix - mod = [(ldap.MOD_ADD, 'krbMKey', str(asn1key))] + #protect the master key by adding an appropriate deny rule along with the key + mod = [(ldap.MOD_ADD, 'aci', ipautil.template_str(KRBMKEY_DENY_ACI, self.sub_dict)), + (ldap.MOD_ADD, 'krbMKey', str(asn1key))] try: self.conn.modify_s(dn, mod) except ldap.TYPE_OR_VALUE_EXISTS, e: |