Skip to content

Commit 75db591

Browse files
siemen11nasahlpa
authored andcommitted
[crypto/hmac] Return from clean
To save some code size, return directly from the clean static function. Signed-off-by: Siemen Dhooghe <sdhooghe@google.com>
1 parent a0f6f56 commit 75db591

1 file changed

Lines changed: 4 additions & 8 deletions

File tree

  • sw/device/lib/crypto/drivers

sw/device/lib/crypto/drivers/hmac.c

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -476,8 +476,7 @@ static status_t oneshot(const uint32_t cfg, const hmac_key_t *key,
476476

477477
HARDENED_CHECK_EQ(kHardenedBoolTrue, OTCRYPTO_CHECK_BUF(msg));
478478

479-
HARDENED_TRY(clear());
480-
return OTCRYPTO_OK;
479+
return clear();
481480
}
482481

483482
/**
@@ -876,12 +875,10 @@ status_t hmac_update(hmac_ctx_t *ctx, const otcrypto_const_byte_buf_t *data) {
876875
leftover_len);
877876
ctx->partial_block_bytelen = leftover_len;
878877

879-
// Clean up.
880-
HARDENED_TRY(clear());
881-
882878
HARDENED_CHECK_EQ(kHardenedBoolTrue, OTCRYPTO_CHECK_BUF(data));
883879

884-
return OTCRYPTO_OK;
880+
// Clean up.
881+
return clear();
885882
}
886883

887884
status_t hmac_final(hmac_ctx_t *ctx, otcrypto_word32_buf_t *digest) {
@@ -912,6 +909,5 @@ status_t hmac_final(hmac_ctx_t *ctx, otcrypto_word32_buf_t *digest) {
912909
HARDENED_CHECK_EQ(kHardenedBoolTrue, OTCRYPTO_CHECK_BUF(digest));
913910

914911
// Clean up.
915-
HARDENED_TRY(clear());
916-
return OTCRYPTO_OK;
912+
return clear();
917913
}

0 commit comments

Comments
 (0)