mirror of
https://github.com/Mbed-TLS/mbedtls.git
synced 2025-07-30 22:43:08 +03:00
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:
@ -514,10 +514,10 @@ exit:
|
|||||||
* where pointers are to the raw certificate, but here all the
|
* where pointers are to the raw certificate, but here all the
|
||||||
* pointers were allocated while parsing from a user-provided string. */
|
* pointers were allocated while parsing from a user-provided string. */
|
||||||
if (cur->node.type == MBEDTLS_X509_SAN_DIRECTORY_NAME) {
|
if (cur->node.type == MBEDTLS_X509_SAN_DIRECTORY_NAME) {
|
||||||
mbedtls_x509_name dn = cur->node.san.directory_name;
|
mbedtls_x509_name *dn = &cur->node.san.directory_name;
|
||||||
mbedtls_free(dn.oid.p);
|
mbedtls_free(dn->oid.p);
|
||||||
mbedtls_free(dn.val.p);
|
mbedtls_free(dn->val.p);
|
||||||
mbedtls_asn1_free_named_data_list(&dn.next);
|
mbedtls_asn1_free_named_data_list(&dn->next);
|
||||||
}
|
}
|
||||||
mbedtls_free(cur);
|
mbedtls_free(cur);
|
||||||
cur = next;
|
cur = next;
|
||||||
|
@ -1008,10 +1008,10 @@ exit:
|
|||||||
* where pointers are to the raw certificate, but here all the
|
* where pointers are to the raw certificate, but here all the
|
||||||
* pointers were allocated while parsing from a user-provided string. */
|
* pointers were allocated while parsing from a user-provided string. */
|
||||||
if (cur->node.type == MBEDTLS_X509_SAN_DIRECTORY_NAME) {
|
if (cur->node.type == MBEDTLS_X509_SAN_DIRECTORY_NAME) {
|
||||||
mbedtls_x509_name dn = cur->node.san.directory_name;
|
mbedtls_x509_name *dn = &cur->node.san.directory_name;
|
||||||
mbedtls_free(dn.oid.p);
|
mbedtls_free(dn->oid.p);
|
||||||
mbedtls_free(dn.val.p);
|
mbedtls_free(dn->val.p);
|
||||||
mbedtls_asn1_free_named_data_list(&dn.next);
|
mbedtls_asn1_free_named_data_list(&dn->next);
|
||||||
}
|
}
|
||||||
mbedtls_free(cur);
|
mbedtls_free(cur);
|
||||||
cur = next;
|
cur = next;
|
||||||
|
Reference in New Issue
Block a user