mirror of
https://github.com/Mbed-TLS/mbedtls.git
synced 2025-03-29 22:20:30 +00:00
pkcs7: Use end_issuer_and_sn where appropriate
There were some areas where `end_signer` were being used when it makes more sense to use `end_issuer_and_sn`, as pointed out by demiobenour@gmail.com. Signed-off-by: Nick Child <nick.child@ibm.com>
This commit is contained in:
parent
a0c15d0fec
commit
3bd17f2f58
@ -317,7 +317,7 @@ static int pkcs7_get_signer_info(unsigned char **p, unsigned char *end,
|
|||||||
/* Parsing IssuerAndSerialNumber */
|
/* Parsing IssuerAndSerialNumber */
|
||||||
signer->issuer_raw.p = *p;
|
signer->issuer_raw.p = *p;
|
||||||
|
|
||||||
asn1_ret = mbedtls_asn1_get_tag(p, end_signer, &len,
|
asn1_ret = mbedtls_asn1_get_tag(p, end_issuer_and_sn, &len,
|
||||||
MBEDTLS_ASN1_CONSTRUCTED | MBEDTLS_ASN1_SEQUENCE);
|
MBEDTLS_ASN1_CONSTRUCTED | MBEDTLS_ASN1_SEQUENCE);
|
||||||
if (asn1_ret != 0) {
|
if (asn1_ret != 0) {
|
||||||
goto out;
|
goto out;
|
||||||
@ -330,7 +330,7 @@ static int pkcs7_get_signer_info(unsigned char **p, unsigned char *end,
|
|||||||
|
|
||||||
signer->issuer_raw.len = *p - signer->issuer_raw.p;
|
signer->issuer_raw.len = *p - signer->issuer_raw.p;
|
||||||
|
|
||||||
ret = mbedtls_x509_get_serial(p, end_signer, &signer->serial);
|
ret = mbedtls_x509_get_serial(p, end_issuer_and_sn, &signer->serial);
|
||||||
if (ret != 0) {
|
if (ret != 0) {
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user