Skip to content

Commit 8985bc7

Browse files
nasahlpaandreaskurth
authored andcommitted
[pentest] Fix OTP wrong memset size
This commit fixes a bug where the wrong memory size was passed to memset. Signed-off-by: Pascal Nasahl <[email protected]> (cherry picked from commit 8bbeee8)
1 parent 07fd56c commit 8985bc7

File tree

1 file changed

+4
-4
lines changed
  • sw/device/tests/penetrationtests/firmware/fi

1 file changed

+4
-4
lines changed

sw/device/tests/penetrationtests/firmware/fi/otp_fi.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ status_t handle_otp_fi_hw_cfg(ujson_t *uj) {
135135
otp_fi_hwcfg_partition_t uj_output;
136136
memset(uj_output.partition_ref, 0, sizeof(uj_output.partition_ref));
137137
memset(uj_output.partition_fi, 0, sizeof(uj_output.partition_fi));
138-
memset(uj_output.data_faulty, false, sizeof(uj_output.partition_fi));
138+
memset(uj_output.data_faulty, false, sizeof(uj_output.data_faulty));
139139
for (uint32_t i = 0; i < kOtpFiHwCfg0Size; i++) {
140140
uj_output.partition_ref[i] = otp_read32_result_hw_cfg_comp[i];
141141
uj_output.partition_fi[i] = otp_read32_result_hw_cfg_fi[i];
@@ -239,7 +239,7 @@ status_t handle_otp_fi_life_cycle(ujson_t *uj) {
239239
otp_fi_lifecycle_partition_t uj_output;
240240
memset(uj_output.partition_ref, 0, sizeof(uj_output.partition_ref));
241241
memset(uj_output.partition_fi, 0, sizeof(uj_output.partition_fi));
242-
memset(uj_output.data_faulty, false, sizeof(uj_output.partition_fi));
242+
memset(uj_output.data_faulty, false, sizeof(uj_output.data_faulty));
243243
for (uint32_t i = 0; i < kOtpFiLifeCycleSize; i++) {
244244
uj_output.partition_ref[i] = otp_read32_result_life_cycle_comp[i];
245245
uj_output.partition_fi[i] = otp_read32_result_life_cycle_fi[i];
@@ -294,7 +294,7 @@ status_t handle_otp_fi_owner_sw_cfg(ujson_t *uj) {
294294
otp_fi_ownerswcfg_partition_t uj_output;
295295
memset(uj_output.partition_ref, 0, sizeof(uj_output.partition_ref));
296296
memset(uj_output.partition_fi, 0, sizeof(uj_output.partition_fi));
297-
memset(uj_output.data_faulty, false, sizeof(uj_output.partition_fi));
297+
memset(uj_output.data_faulty, false, sizeof(uj_output.data_faulty));
298298
for (uint32_t i = 0; i < kOtpFiOwnerSwCfgSize; i++) {
299299
uj_output.partition_ref[i] = otp_read32_result_owner_sw_cfg_comp[i];
300300
uj_output.partition_fi[i] = otp_read32_result_owner_sw_cfg_fi[i];
@@ -349,7 +349,7 @@ status_t handle_otp_fi_vendor_test(ujson_t *uj) {
349349
otp_fi_vendortest_partition_t uj_output;
350350
memset(uj_output.partition_ref, 0, sizeof(uj_output.partition_ref));
351351
memset(uj_output.partition_fi, 0, sizeof(uj_output.partition_fi));
352-
memset(uj_output.data_faulty, false, sizeof(uj_output.partition_fi));
352+
memset(uj_output.data_faulty, false, sizeof(uj_output.data_faulty));
353353
for (uint32_t i = 0; i < kOtpFiVendorTestSize; i++) {
354354
uj_output.partition_ref[i] = otp_read32_result_vendor_test_comp[i];
355355
uj_output.partition_fi[i] = otp_read32_result_vendor_test_fi[i];

0 commit comments

Comments
 (0)