yuzu-emu/mbedtls
1
0
Fork 0
mirror of https://github.com/yuzu-emu/mbedtls.git synced 2024-11-25 19:45:44 +01:00
Code Issues Packages Projects Releases Wiki Activity

Update after feedback on #3492

Browse Source 
* Updated wording
* Split out buffer allocation to a convenience function
* Moved variable declarations to beginning of their code block

Signed-off-by: Steven Cooreman <steven.cooreman@silabs.com>
This commit is contained in:
Steven Cooreman 2020-07-28 14:30:13 +02:00
parent 19fd574b3a
commit 75b743666e
1 changed files with 58 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

95
library/psa_crypto.c
View File

@ -605,8 +605,8 @@ static psa_status_t psa_export_rsa_key( psa_key_type_t type,
pk.pk_ctx = rsa; pk.pk_ctx = rsa;
/* PSA Crypto API defines the format of an RSA key as a DER-encoded /* PSA Crypto API defines the format of an RSA key as a DER-encoded
* representation of respectively the non-encrypted PKCS#1 RSAPrivateKey * representation of the non-encrypted PKCS#1 RSAPrivateKey for a
* or the RFC3279 RSAPublicKey for a private key or a public key. */ * private key and of the RFC3279 RSAPublicKey for a public key. */
if( PSA_KEY_TYPE_IS_KEY_PAIR( type ) ) if( PSA_KEY_TYPE_IS_KEY_PAIR( type ) )
ret = mbedtls_pk_write_key_der( &pk, data, data_size ); ret = mbedtls_pk_write_key_der( &pk, data, data_size );
else else
@ -670,8 +670,10 @@ static psa_status_t psa_import_rsa_key( psa_key_slot_t *slot,
slot->attr.bits = (psa_key_bits_t) PSA_BYTES_TO_BITS( slot->attr.bits = (psa_key_bits_t) PSA_BYTES_TO_BITS(
mbedtls_rsa_get_len( &rsa ) ); mbedtls_rsa_get_len( &rsa ) );
/* Re-export the data to PSA export format, which in case of RSA is the /* Re-export the data to PSA export format, such that we can store export
* smallest representation we can parse. */ * representation in the key slot. Export representation in case of RSA is
* the smallest representation that's allowed as input, so a straight-up
* allocation of the same size as the input buffer will be large enough. */
output = mbedtls_calloc( 1, data_length ); output = mbedtls_calloc( 1, data_length );
if( output == NULL ) if( output == NULL )
@ -680,11 +682,6 @@ static psa_status_t psa_import_rsa_key( psa_key_slot_t *slot,
goto exit; goto exit;
} }
/* PSA Crypto API defines the format of an RSA key as a DER-encoded
* representation of respectively the non-encrypted PKCS#1 RSAPrivateKey
* or the RFC3279 RSAPublicKey for a private key or a public key. That
* means we have no other choice then to run an import to verify the key
* size. */
status = psa_export_rsa_key( slot->attr.type, status = psa_export_rsa_key( slot->attr.type,
&rsa, &rsa,
output, output,
@ -905,6 +902,32 @@ static inline size_t psa_get_key_slot_bits( const psa_key_slot_t *slot )
return( slot->attr.bits ); return( slot->attr.bits );
} }
/** Try to allocate a buffer to an empty key slot.
*
* \param[in,out] slot Key slot to attach buffer to.
* \param[in] buffer_length Requested size of the buffer.
*
* \retval #PSA_SUCCESS
* The buffer has been successfully allocated.
* \retval #PSA_ERROR_INSUFFICIENT_MEMORY
* Not enough memory was available for allocation.
* \retval #PSA_ERROR_ALREADY_EXISTS
* Trying to allocate a buffer to a non-empty key slot.
*/
static psa_status_t psa_allocate_buffer_to_slot( psa_key_slot_t *slot,
size_t buffer_length )
{
if( slot->data.key.data != NULL )
return PSA_ERROR_ALREADY_EXISTS;
slot->data.key.data = mbedtls_calloc( 1, buffer_length );
if( slot->data.key.data == NULL )
return PSA_ERROR_INSUFFICIENT_MEMORY;
slot->data.key.bytes = buffer_length;
return PSA_SUCCESS;
}
/** Import key data into a slot. `slot->attr.type` must have been set /** Import key data into a slot. `slot->attr.type` must have been set
* previously. This function assumes that the slot does not contain * previously. This function assumes that the slot does not contain
* any key material yet. On failure, the slot content is unchanged. */ * any key material yet. On failure, the slot content is unchanged. */
@ -918,14 +941,14 @@ psa_status_t psa_import_key_into_slot( psa_key_slot_t *slot,
if( data_length == 0 ) if( data_length == 0 )
return( PSA_ERROR_NOT_SUPPORTED ); return( PSA_ERROR_NOT_SUPPORTED );
/* Ensure that the bytes-to-bit conversion never overflows. */
if( data_length > SIZE_MAX / 8 )
return( PSA_ERROR_NOT_SUPPORTED );
if( key_type_is_raw_bytes( slot->attr.type ) ) if( key_type_is_raw_bytes( slot->attr.type ) )
{ {
size_t bit_size = PSA_BYTES_TO_BITS( data_length ); size_t bit_size = PSA_BYTES_TO_BITS( data_length );
/* Ensure that the bytes-to-bits conversion hasn't overflown. */
if( data_length > SIZE_MAX / 8 )
return( PSA_ERROR_NOT_SUPPORTED );
/* Enforce a size limit, and in particular ensure that the bit /* Enforce a size limit, and in particular ensure that the bit
* size fits in its representation type. */ * size fits in its representation type. */
if( bit_size > PSA_MAX_KEY_BITS ) if( bit_size > PSA_MAX_KEY_BITS )
@ -936,12 +959,9 @@ psa_status_t psa_import_key_into_slot( psa_key_slot_t *slot,
return status; return status;
/* Allocate memory for the key */ /* Allocate memory for the key */
slot->data.key.data = mbedtls_calloc( 1, data_length ); status = psa_allocate_buffer_to_slot( slot, data_length );
if( slot->data.key.data == NULL ) if( status != PSA_SUCCESS )
{ return status;
return( PSA_ERROR_INSUFFICIENT_MEMORY );
}
slot->data.key.bytes = data_length;
/* copy key into allocated buffer */ /* copy key into allocated buffer */
memcpy(slot->data.key.data, data, data_length); memcpy(slot->data.key.data, data, data_length);
@ -1135,6 +1155,10 @@ static psa_status_t psa_get_transparent_key( psa_key_handle_t handle,
/** Wipe key data from a slot. Preserve metadata such as the policy. */ /** Wipe key data from a slot. Preserve metadata such as the policy. */
static psa_status_t psa_remove_key_data_from_memory( psa_key_slot_t *slot ) static psa_status_t psa_remove_key_data_from_memory( psa_key_slot_t *slot )
{ {
/* Check whether key is already clean */
if( slot->data.key.data == NULL )
return PSA_SUCCESS;
#if defined(MBEDTLS_PSA_CRYPTO_SE_C) #if defined(MBEDTLS_PSA_CRYPTO_SE_C)
if( psa_key_slot_is_external( slot ) ) if( psa_key_slot_is_external( slot ) )
{ {
@ -1958,11 +1982,12 @@ static psa_status_t psa_validate_optional_attributes(
{ {
mbedtls_rsa_context rsa; mbedtls_rsa_context rsa;
mbedtls_rsa_init( &rsa, MBEDTLS_RSA_PKCS_V15, MBEDTLS_MD_NONE ); mbedtls_rsa_init( &rsa, MBEDTLS_RSA_PKCS_V15, MBEDTLS_MD_NONE );
mbedtls_mpi actual, required;
psa_status_t status = psa_load_rsa_representation( slot, &rsa ); psa_status_t status = psa_load_rsa_representation( slot, &rsa );
if( status != PSA_SUCCESS ) if( status != PSA_SUCCESS )
return status; return status;
mbedtls_mpi actual, required;
int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED;
mbedtls_mpi_init( &actual ); mbedtls_mpi_init( &actual );
mbedtls_mpi_init( &required ); mbedtls_mpi_init( &required );
@ -3808,11 +3833,11 @@ psa_status_t psa_asymmetric_encrypt( psa_key_handle_t handle,
{ {
mbedtls_rsa_context rsa; mbedtls_rsa_context rsa;
mbedtls_rsa_init( &rsa, MBEDTLS_RSA_PKCS_V15, MBEDTLS_MD_NONE ); mbedtls_rsa_init( &rsa, MBEDTLS_RSA_PKCS_V15, MBEDTLS_MD_NONE );
int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED;
status = psa_load_rsa_representation( slot, &rsa ); status = psa_load_rsa_representation( slot, &rsa );
if( status != PSA_SUCCESS ) if( status != PSA_SUCCESS )
return status; return status;
int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED;
if( output_size < mbedtls_rsa_get_len( &rsa ) ) if( output_size < mbedtls_rsa_get_len( &rsa ) )
{ {
mbedtls_rsa_free( &rsa ); mbedtls_rsa_free( &rsa );
@ -3898,11 +3923,11 @@ psa_status_t psa_asymmetric_decrypt( psa_key_handle_t handle,
{ {
mbedtls_rsa_context rsa; mbedtls_rsa_context rsa;
mbedtls_rsa_init( &rsa, MBEDTLS_RSA_PKCS_V15, MBEDTLS_MD_NONE ); mbedtls_rsa_init( &rsa, MBEDTLS_RSA_PKCS_V15, MBEDTLS_MD_NONE );
int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED;
status = psa_load_rsa_representation( slot, &rsa ); status = psa_load_rsa_representation( slot, &rsa );
if( status != PSA_SUCCESS ) if( status != PSA_SUCCESS )
return status; return status;
int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED;
if( input_length != mbedtls_rsa_get_len( &rsa ) ) if( input_length != mbedtls_rsa_get_len( &rsa ) )
{ {
@ -5773,13 +5798,9 @@ static psa_status_t psa_generate_key_internal(
return( status ); return( status );
/* Allocate memory for the key */ /* Allocate memory for the key */
slot->data.key.bytes = PSA_BITS_TO_BYTES( bits ); status = psa_allocate_buffer_to_slot( slot, PSA_BITS_TO_BYTES( bits ) );
slot->data.key.data = mbedtls_calloc( 1, slot->data.key.bytes ); if( status != PSA_SUCCESS )
if( slot->data.key.data == NULL ) return status;
{
slot->data.key.bytes = 0;
return( PSA_ERROR_INSUFFICIENT_MEMORY );
}
status = psa_generate_random( slot->data.key.data, status = psa_generate_random( slot->data.key.data,
slot->data.key.bytes ); slot->data.key.bytes );
@ -5825,11 +5846,11 @@ static psa_status_t psa_generate_key_internal(
/* Make sure to always have an export representation available */ /* Make sure to always have an export representation available */
size_t bytes = PSA_KEY_EXPORT_RSA_KEY_PAIR_MAX_SIZE( bits ); size_t bytes = PSA_KEY_EXPORT_RSA_KEY_PAIR_MAX_SIZE( bits );
slot->data.key.data = mbedtls_calloc( 1, bytes ); status = psa_allocate_buffer_to_slot( slot, bytes );
if( slot->data.key.data == NULL ) if( status != PSA_SUCCESS )
{ {
mbedtls_rsa_free( &rsa ); mbedtls_rsa_free( &rsa );
return( PSA_ERROR_INSUFFICIENT_MEMORY ); return status;
} }
status = psa_export_rsa_key( type, status = psa_export_rsa_key( type,
@ -5874,14 +5895,14 @@ static psa_status_t psa_generate_key_internal(
/* Make sure to always have an export representation available */ /* Make sure to always have an export representation available */
size_t bytes = PSA_BITS_TO_BYTES( bits ); size_t bytes = PSA_BITS_TO_BYTES( bits );
slot->data.key.data = mbedtls_calloc( 1, bytes ); psa_status_t status = psa_allocate_buffer_to_slot( slot, bytes );
if( slot->data.key.data == NULL ) if( status != PSA_SUCCESS )
{ {
mbedtls_ecp_keypair_free( &ecp ); mbedtls_ecp_keypair_free( &ecp );
return( PSA_ERROR_INSUFFICIENT_MEMORY ); return status;
} }
slot->data.key.bytes = bytes;
psa_status_t status = mbedtls_to_psa_error( status = mbedtls_to_psa_error(
mbedtls_ecp_write_key( &ecp, slot->data.key.data, bytes ) ); mbedtls_ecp_write_key( &ecp, slot->data.key.data, bytes ) );
mbedtls_ecp_keypair_free( &ecp ); mbedtls_ecp_keypair_free( &ecp );