diff options
| author | Rob Austein <sra@hactrn.net> | 2008-05-07 12:34:43 +0000 |
|---|---|---|
| committer | Rob Austein <sra@hactrn.net> | 2008-05-07 12:34:43 +0000 |
| commit | da4b762da0c8b2efe0923d5d46adac33b9a13313 (patch) | |
| tree | 1ffd012ca5827239469aace004c1837d994d293b | |
| parent | dbd6286dbe2b8607a7b46569753de2cb55a9a8cb (diff) | |
Checkpoint
svn path=/rpkid/rpki/exceptions.py; revision=1748
| -rw-r--r-- | rpkid/rpki/exceptions.py | 3 | ||||
| -rw-r--r-- | rpkid/rpki/x509.py | 18 |
2 files changed, 18 insertions, 3 deletions
diff --git a/rpkid/rpki/exceptions.py b/rpkid/rpki/exceptions.py index 8bde9cde..a9579337 100644 --- a/rpkid/rpki/exceptions.py +++ b/rpkid/rpki/exceptions.py @@ -111,3 +111,6 @@ class MissingCMSEEcert(RPKI_Exception): class MissingCMSCRL(RPKI_Exception): """Didn't receive CMS CRL when expecting one.""" + +class UnparsableCMSDER(RPKI_Exception): + """Alleged CMS DER wasn't parsable.""" diff --git a/rpkid/rpki/x509.py b/rpkid/rpki/x509.py index d39e6093..3aadab05 100644 --- a/rpkid/rpki/x509.py +++ b/rpkid/rpki/x509.py @@ -580,7 +580,7 @@ class CMS_object(DER_object): ## @var dump_on_verify_failure # Set this to True to get dumpasn1 dumps of ASN.1 on CMS verify failures. - dump_on_verify_failure = False + dump_on_verify_failure = True ## @var debug_cms_certs # Set this to True to log a lot of chatter about CMS certificates. @@ -594,6 +594,12 @@ class CMS_object(DER_object): require_crls = False + ## @var print_on_der_error + # Log alleged DER when we have trouble parsing it, in case it's + # really somebody's Perl debug trace or something. + + print_on_der_error = True + def get_DER(self): """Get the DER value of this CMS_object.""" assert not self.empty() @@ -614,10 +620,16 @@ class CMS_object(DER_object): def verify(self, ta): """Verify CMS wrapper and store inner content.""" - cms = POW.derRead(POW.CMS_MESSAGE, self.get_DER()) + try: + cms = POW.derRead(POW.CMS_MESSAGE, self.get_DER()) + except: + if self.print_on_der_error: + rpki.log.debug("Problem parsing DER CMS message, might not really be DER: %s" + % repr(self.get_DER())) + raise rpki.exceptions.UnparsableCMSDER, self if cms.eContentType() != self.econtent_oid: - raise rpki.exceptions.WrongEContentType, "Got CMS eContentType %s, expected %s" % (cms.eContentType(), self.econtent_oid) + raise rpki.exceptions.WrongEContentType, "Got CMS eContentType %s, expected %s" % (cms.eContentType(), self.econtent_oid), cms certs = [X509(POW = x) for x in cms.certs()] crls = [CRL(POW = c) for c in cms.crls()] |