commit 3015148ae6ae89d19947c66136d5b005ecf8b2e3
author pespacek <peter.spacek@silabs.com> Thu Feb 17 15:18:47 2022 +0100
committer pespacek <peter.spacek@silabs.com> Thu Feb 17 16:08:23 2022 +0100
tree 2e3009f5f291361d4f8616cd38e3267fd715bc0a
parent a6e955e7296ff1e0fdda6a1bf9a0864b31ae3bc0

Improving readability

Signed-off-by: pespacek <peter.spacek@silabs.com>

library/x509write_crt.c

diff --git a/library/x509write_crt.c b/library/x509write_crt.c
index be73876..07f5851 100644
--- a/library/x509write_crt.c
+++ b/library/x509write_crt.c
@@ -187,27 +187,19 @@
 #endif /* MBEDTLS_USE_PSA_CRYPTO */
 
     memset( buf, 0, sizeof(buf) );
-    if( is_ca )
-    {
-        MBEDTLS_ASN1_CHK_ADD( len,
-                              mbedtls_pk_write_pubkey( &c,
-                                                       buf,
-                                                       ctx->issuer_key ) );
-    }
-    else
-    {
-        MBEDTLS_ASN1_CHK_ADD( len,
-                              mbedtls_pk_write_pubkey( &c,
-                                                       buf,
-                                                       ctx->subject_key ) );
-    }
+    MBEDTLS_ASN1_CHK_ADD( len, mbedtls_pk_write_pubkey( &c,
+                                                        buf,
+                                                        is_ca ?
+                                                          ctx->issuer_key :
+                                                          ctx->subject_key ) );
+
 
 #if defined(MBEDTLS_USE_PSA_CRYPTO)
     status = psa_hash_compute( PSA_ALG_SHA_1,
                                buf + sizeof(buf) - len,
                                len,
                                buf + sizeof(buf) - 20,
-                               PSA_HASH_LENGTH(PSA_ALG_SHA_1),
+                               20,
                                &hash_length );
     if( status != PSA_SUCCESS )
     {
@@ -235,24 +227,18 @@
                                                       buf,
                                                       MBEDTLS_ASN1_CONSTRUCTED |
                                                       MBEDTLS_ASN1_SEQUENCE ) );
-        return mbedtls_x509write_crt_set_extension(
+    }
+    return mbedtls_x509write_crt_set_extension(
                    ctx,
-                   MBEDTLS_OID_AUTHORITY_KEY_IDENTIFIER,
-                   MBEDTLS_OID_SIZE( MBEDTLS_OID_AUTHORITY_KEY_IDENTIFIER ),
+                   is_ca ? MBEDTLS_OID_AUTHORITY_KEY_IDENTIFIER :
+                               MBEDTLS_OID_SUBJECT_KEY_IDENTIFIER,
+                   is_ca ? MBEDTLS_OID_SIZE(
+                               MBEDTLS_OID_AUTHORITY_KEY_IDENTIFIER ) :
+                           MBEDTLS_OID_SIZE(
+                               MBEDTLS_OID_SUBJECT_KEY_IDENTIFIER ),
                    0,
                    buf + sizeof( buf ) - len,
                    len );
-    }
-    else
-    {
-        return mbedtls_x509write_crt_set_extension(
-                   ctx,
-                   MBEDTLS_OID_SUBJECT_KEY_IDENTIFIER,
-                   MBEDTLS_OID_SIZE( MBEDTLS_OID_SUBJECT_KEY_IDENTIFIER ),
-                   0,
-                   buf + sizeof( buf ) - len,
-                   len );
-    }
 }
 
 int mbedtls_x509write_crt_set_subject_key_identifier( mbedtls_x509write_cert *ctx )