From 442973edc5bffc1e215c22327369f66df7aeec08 Mon Sep 17 00:00:00 2001 From: John Dennis Date: Thu, 28 Jul 2011 14:32:26 -0400 Subject: Clean up existing DN object usage --- ipaserver/install/certs.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) (limited to 'ipaserver/install/certs.py') diff --git a/ipaserver/install/certs.py b/ipaserver/install/certs.py index 1bbcbabe6..738b329a6 100644 --- a/ipaserver/install/certs.py +++ b/ipaserver/install/certs.py @@ -89,13 +89,15 @@ def get_cert_nickname(cert): for NSS. The caller can decide whether to use just the RDN or the whole subject. - Returns a tuple of (rdn, subject) + Returns a tuple of (rdn, subject_dn) when rdn is the string + representation of the first RDN in the subject and subject_dn + is a DN object. """ nsscert = x509.load_certificate(cert) subject = str(nsscert.subject) dn = DN(subject) - return (str(dn[0]), str(dn)) + return (str(dn[0]), dn) def next_serial(serial_file=CA_SERIALNO): """ @@ -430,16 +432,16 @@ class CertDB(object): certs = fd.read() fd.close() - normalized_base = str(DN(self.subject_base)) + ca_dn = DN(('CN','Certificate Authority'), self.subject_base) st = 0 while True: try: (cert, st) = find_cert_from_txt(certs, st) - (nick, subject) = get_cert_nickname(cert) - if subject.lower() == ('CN=Certificate Authority,%s' % normalized_base).lower(): + (rdn, subject_dn) = get_cert_nickname(cert) + if subject_dn == ca_dn: nick = get_ca_nickname(self.realm) else: - nick = subject + nick = str(subject_dn) self.run_certutil(["-A", "-n", nick, "-t", "CT,,C", "-a"], -- cgit