Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fi] Add OTBN command handlers #338

Merged
merged 1 commit into from
Mar 21, 2024
Merged

Conversation

nasahlpa
Copy link
Member

This commit adds command handlers for the following tests:

  • otbn_key_sideload
  • otbn_load_integrity

The device code is located in lowRISC/opentitan#21917. The binary was created with this PR.

@nasahlpa nasahlpa requested review from vogelpi and vrozic as code owners March 13, 2024 15:27
nasahlpa added a commit to nasahlpa/opentitan that referenced this pull request Mar 13, 2024
This commits adds the following two FI penetration tests for OTBN:
- otbn.fi.load_integrity
- otbn.fi.key_sideload

The first test aims to check whether the CRC checksum over the
DMEM can be manipulated.
The second test aims to test whether a fault can manipulate a key
that gets sideloaded from the key manager.

The host code is located in lowRISC/ot-sca#338

Signed-off-by: Pascal Nasahl <[email protected]>
nasahlpa added a commit to nasahlpa/opentitan that referenced this pull request Mar 14, 2024
This commits adds the following two FI penetration tests for OTBN:
- otbn.fi.load_integrity
- otbn.fi.key_sideload

The first test aims to check whether the CRC checksum over the
DMEM can be manipulated.
The second test aims to test whether a fault can manipulate a key
that gets sideloaded from the key manager.

The host code is located in lowRISC/ot-sca#338

Signed-off-by: Pascal Nasahl <[email protected]>
Copy link
Collaborator

@johannheyszl johannheyszl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @nasahlpa

moidx pushed a commit to lowRISC/opentitan that referenced this pull request Mar 21, 2024
This commits adds the following two FI penetration tests for OTBN:
- otbn.fi.load_integrity
- otbn.fi.key_sideload

The first test aims to check whether the CRC checksum over the
DMEM can be manipulated.
The second test aims to test whether a fault can manipulate a key
that gets sideloaded from the key manager.

The host code is located in lowRISC/ot-sca#338

Signed-off-by: Pascal Nasahl <[email protected]>
github-actions bot pushed a commit to lowRISC/opentitan that referenced this pull request Mar 21, 2024
This commits adds the following two FI penetration tests for OTBN:
- otbn.fi.load_integrity
- otbn.fi.key_sideload

The first test aims to check whether the CRC checksum over the
DMEM can be manipulated.
The second test aims to test whether a fault can manipulate a key
that gets sideloaded from the key manager.

The host code is located in lowRISC/ot-sca#338

Signed-off-by: Pascal Nasahl <[email protected]>
(cherry picked from commit 3be00e3)
This commit adds command handlers for the following tests:
- otbn_key_sideload
- otbn_load_integrity

The device code is located in lowRISC/opentitan#21917. The
binary was created from lowRISC/opentitan@55091e7 with
./bazelisk.sh build //sw/device/tests/crypto/cryptotest/firmware:firmware_fpga_cw310_test_rom

Signed-off-by: Pascal Nasahl <[email protected]>
@nasahlpa nasahlpa merged commit 679d99f into lowRISC:master Mar 21, 2024
8 checks passed
@nasahlpa nasahlpa deleted the otbn_fi_tests branch March 21, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants