- Added a separate CRL entry extension parsing function

This commit is contained in:
Paul Bakker 2011-10-12 09:58:41 +00:00
parent fbc09f3cb6
commit b5a11ab80b
2 changed files with 60 additions and 3 deletions

View File

@ -15,6 +15,7 @@ Changes
encryption and private key for decryption. (Closes ticket #34) encryption and private key for decryption. (Closes ticket #34)
* Inceased maximum size of ASN1 length reads to 32-bits. * Inceased maximum size of ASN1 length reads to 32-bits.
* Added an EXPLICIT tag number parameter to x509_get_ext() * Added an EXPLICIT tag number parameter to x509_get_ext()
* Added a separate CRL entry extension parsing function
Bugfix Bugfix
* Fixed faulty HMAC-MD2 implementation. Found by dibac. (Closes * Fixed faulty HMAC-MD2 implementation. Found by dibac. (Closes

View File

@ -827,6 +827,60 @@ static int x509_get_crl_ext( unsigned char **p,
return( 0 ); return( 0 );
} }
/*
* X.509 CRL v2 entry extensions (no extensions parsed yet.)
*/
static int x509_get_crl_entry_ext( unsigned char **p,
const unsigned char *end,
x509_buf *ext )
{
int ret;
size_t len = 0;
/* OPTIONAL */
if (end <= *p)
return( 0 );
ext->tag = **p;
ext->p = *p;
/*
* Get CRL-entry extension sequence header
* crlEntryExtensions Extensions OPTIONAL -- if present, MUST be v2
*/
if( ( ret = asn1_get_tag( p, end, &ext->len,
ASN1_CONSTRUCTED | ASN1_SEQUENCE ) ) != 0 )
{
if( ret == POLARSSL_ERR_ASN1_UNEXPECTED_TAG )
{
ext->p = NULL;
return( 0 );
}
return( POLARSSL_ERR_X509_CERT_INVALID_EXTENSIONS + ret );
}
end = *p + ext->len;
if( end != *p + ext->len )
return( POLARSSL_ERR_X509_CERT_INVALID_EXTENSIONS +
POLARSSL_ERR_ASN1_LENGTH_MISMATCH );
while( *p < end )
{
if( ( ret = asn1_get_tag( p, end, &len,
ASN1_CONSTRUCTED | ASN1_SEQUENCE ) ) != 0 )
return( POLARSSL_ERR_X509_CERT_INVALID_EXTENSIONS + ret );
*p += len;
}
if( *p != end )
return( POLARSSL_ERR_X509_CERT_INVALID_EXTENSIONS +
POLARSSL_ERR_ASN1_LENGTH_MISMATCH );
return( 0 );
}
static int x509_get_basic_constraints( unsigned char **p, static int x509_get_basic_constraints( unsigned char **p,
const unsigned char *end, const unsigned char *end,
int *ca_istrue, int *ca_istrue,
@ -1096,6 +1150,7 @@ static int x509_get_entries( unsigned char **p,
while( *p < end ) while( *p < end )
{ {
size_t len2; size_t len2;
const unsigned char *end2;
if( ( ret = asn1_get_tag( p, end, &len2, if( ( ret = asn1_get_tag( p, end, &len2,
ASN1_SEQUENCE | ASN1_CONSTRUCTED ) ) != 0 ) ASN1_SEQUENCE | ASN1_CONSTRUCTED ) ) != 0 )
@ -1106,14 +1161,15 @@ static int x509_get_entries( unsigned char **p,
cur_entry->raw.tag = **p; cur_entry->raw.tag = **p;
cur_entry->raw.p = *p; cur_entry->raw.p = *p;
cur_entry->raw.len = len2; cur_entry->raw.len = len2;
end2 = *p + len2;
if( ( ret = x509_get_serial( p, end, &cur_entry->serial ) ) != 0 ) if( ( ret = x509_get_serial( p, end2, &cur_entry->serial ) ) != 0 )
return( ret ); return( ret );
if( ( ret = x509_get_time( p, end, &cur_entry->revocation_date ) ) != 0 ) if( ( ret = x509_get_time( p, end2, &cur_entry->revocation_date ) ) != 0 )
return( ret ); return( ret );
if( ( ret = x509_get_crl_ext( p, end, &cur_entry->entry_ext ) ) != 0 ) if( ( ret = x509_get_crl_entry_ext( p, end2, &cur_entry->entry_ext ) ) != 0 )
return( ret ); return( ret );
if ( *p < end ) if ( *p < end )