Skip to content
This repository was archived by the owner on Feb 25, 2023. It is now read-only.

Add HSM version file / Redis key #315

Merged
merged 1 commit into from
Dec 12, 2019

Conversation

Stadicus
Copy link
Collaborator

related: #314

To check if the HSM firmware should be updated, the Base image needs to know the firmware version that it contains. From the firmware file, only the versioning counter can be read directly, but to get this information from the HSM it would need to boot into the bootloader first, which is cumbersome.

Getting the semantic version from the HSM firmware (not bootloader) and decide whether to update is less complex.

The downgrade protection using the monotonic counter is still in place when actually updating the HSM firmware.

This commit:

  • adds the file config/version_hsm, containing the semantiv version
  • renames the file config/version to config/version_bbb for consistency
  • stores the HSM firmware version in Redis key hsm:firmware:version during build
  • resets this Redis key from the file on every boot

Verified

This commit was signed with the committer’s verified signature. The key has expired.
Stadicus Stadicus
Because:
* To check if the HSM firmware should be updated, the Base image needs
  to know the firmware version that it contains.
* From the firmware file, only the versioning counter can be read
  directly, but to get this information from the HSM it would need to
  boot into the bootloader first, which is cumbersome.
* Getting the semantic version from the HSM firmware (not bootloader)
  and decide whether to update is less complex.
* The downgrade protection using the monotonic counter is still in
  place when actually updating the HSM firmware.

This commit:
* adds the file `config/version_hsm`, containing the semantiv version
* renames the file `config/version` to `config/version_bbb` for
  consistency
* stores the HSM firmware version in Redis key `hsm:firmware:version`
  during build
* resets this Redis key from the file on every boot
@Stadicus Stadicus requested a review from Tomasvrba December 12, 2019 13:02
@Stadicus Stadicus self-assigned this Dec 12, 2019
Copy link
Contributor

@Tomasvrba Tomasvrba left a comment

Choose a reason for hiding this comment

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

utACK

@Stadicus Stadicus merged commit 60ffdee into BitBoxSwiss:master Dec 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants