diff options
author | Demi Marie Obenour <demiobenour@gmail.com> | 2022-12-08 15:24:36 -0500 |
---|---|---|
committer | Sandrine Bailleux <sandrine.bailleux@arm.com> | 2023-01-03 17:49:24 +0100 |
commit | ce882b53649f754f92e0e7e42fee81ffd9561d53 (patch) | |
tree | c98dcf692302b8f93b622f329afd1db7fc30b723 /drivers/auth | |
parent | ca34dbc0cdb1c4e1ab62aa4dd195cf9389b9edb7 (diff) | |
download | arm-trusted-firmware-ce882b53649f754f92e0e7e42fee81ffd9561d53.tar.gz |
refactor(auth): do not include SEQUENCE tag in saved extensions
This makes the code a little bit smaller. No functional change
intended.
Change-Id: I794d2927fcd034a79e29c9bba1f8e4410203f547
Signed-off-by: Demi Marie Obenour <demiobenour@gmail.com>
Diffstat (limited to 'drivers/auth')
-rw-r--r-- | drivers/auth/mbedtls/mbedtls_x509_parser.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/auth/mbedtls/mbedtls_x509_parser.c b/drivers/auth/mbedtls/mbedtls_x509_parser.c index d8f201e98..4b880d99c 100644 --- a/drivers/auth/mbedtls/mbedtls_x509_parser.c +++ b/drivers/auth/mbedtls/mbedtls_x509_parser.c @@ -85,9 +85,6 @@ static int get_ext(const char *oid, void **ext, unsigned int *ext_len) p = v3_ext.p; end = v3_ext.p + v3_ext.len; - mbedtls_asn1_get_tag(&p, end, &len, MBEDTLS_ASN1_CONSTRUCTED | - MBEDTLS_ASN1_SEQUENCE); - while (p < end) { zeromem(&extn_oid, sizeof(extn_oid)); is_critical = 0; /* DEFAULT FALSE */ @@ -274,6 +271,7 @@ static int cert_parse(void *img, unsigned int img_len) pk_end = p + len; pk.len = pk_end - pk.p; + /* algorithm */ ret = mbedtls_asn1_get_tag(&p, pk_end, &len, MBEDTLS_ASN1_CONSTRUCTED | MBEDTLS_ASN1_SEQUENCE); if (ret != 0) { @@ -342,13 +340,13 @@ static int cert_parse(void *img, unsigned int img_len) * Extensions ::= SEQUENCE SIZE (1..MAX) OF Extension * -- must use all remaining bytes in TBSCertificate */ - v3_ext.p = p; ret = mbedtls_asn1_get_tag(&p, end, &len, MBEDTLS_ASN1_CONSTRUCTED | MBEDTLS_ASN1_SEQUENCE); if ((ret != 0) || (len != (size_t)(end - p))) { return IMG_PARSER_ERR_FORMAT; } - v3_ext.len = end - v3_ext.p; + v3_ext.p = p; + v3_ext.len = len; /* * Check extensions integrity. At least one extension is |