Avoid a useless copy in cert_{req,write}

I'm just trying to have a shorter name to avoid repeating a long
expression. This is a job for a pointer, not copying a struct.

Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
This commit is contained in:
Manuel Pégourié-Gonnard 2025-05-21 11:17:39 +02:00
parent 8429619a92
commit 8ac3eb9833
2 changed files with 8 additions and 8 deletions

View File

@ -516,10 +516,10 @@ exit:
* where pointers are to the raw certificate, but here all the
* pointers were allocated while parsing from a user-provided string. */
if (cur->node.type == MBEDTLS_X509_SAN_DIRECTORY_NAME) {
mbedtls_x509_name dn = cur->node.san.directory_name;
mbedtls_free(dn.oid.p);
mbedtls_free(dn.val.p);
mbedtls_asn1_free_named_data_list(&dn.next);
mbedtls_x509_name *dn = &cur->node.san.directory_name;
mbedtls_free(dn->oid.p);
mbedtls_free(dn->val.p);
mbedtls_asn1_free_named_data_list(&dn->next);
}
mbedtls_free(cur);
cur = next;

View File

@ -1012,10 +1012,10 @@ exit:
* where pointers are to the raw certificate, but here all the
* pointers were allocated while parsing from a user-provided string. */
if (cur->node.type == MBEDTLS_X509_SAN_DIRECTORY_NAME) {
mbedtls_x509_name dn = cur->node.san.directory_name;
mbedtls_free(dn.oid.p);
mbedtls_free(dn.val.p);
mbedtls_asn1_free_named_data_list(&dn.next);
mbedtls_x509_name *dn = &cur->node.san.directory_name;
mbedtls_free(dn->oid.p);
mbedtls_free(dn->val.p);
mbedtls_asn1_free_named_data_list(&dn->next);
}
mbedtls_free(cur);
cur = next;