Skip to content

Commit de45e87

Browse files
nasahlpavogelpi
authored andcommitted
[aes,sca] Fix AES-GCM PTX measurement
Accidentally, trigger[3] was used for the PTX as well as for the TAG measurements, causing trigger overlaps. This commit uses trigger[4] for the TAG measurement, as it should be. Signed-off-by: Pascal Nasahl <[email protected]>
1 parent 284ec94 commit de45e87

File tree

1 file changed

+1
-1
lines changed
  • sw/device/tests/penetrationtests/firmware/sca

1 file changed

+1
-1
lines changed

sw/device/tests/penetrationtests/firmware/sca/aes_sca.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -561,7 +561,7 @@ static status_t trigger_aes_gcm(dif_aes_key_share_t key, dif_aes_iv_t iv,
561561
AES_TESTUTILS_WAIT_FOR_STATUS(&aes, kDifAesStatusInputReady, true,
562562
kIbexAesGcmSleepCycles * 2);
563563
TRY(dif_aes_load_gcm_tag_len(&aes, len_ptx, len_aad));
564-
if (trigger.triggers[3]) {
564+
if (trigger.triggers[4]) {
565565
// In the FPGA mode, the AES automatically raises the trigger signal. For
566566
// the other mode, the pentest_call_and_sleep function manually raises the
567567
// trigger pin.

0 commit comments

Comments
 (0)