mbox series

[v2,0/3] ima-evm-utils: Add --keyid option

Message ID 20210504043357.4093409-1-vt@altlinux.org (mailing list archive)
Headers show
Series ima-evm-utils: Add --keyid option | expand

Message

Vitaly Chikunov May 4, 2021, 4:33 a.m. UTC
Allow user to set signature's keyid using `--keyid' option. Keyid should
correspond to SKID in certificate, when keyid is calculated using SHA-1
in libimaevm it may mismatch keyid extracted by the kernel from SKID of
certificate (the way public key is presented to the kernel), thus making
signatures not verifiable. This may happen when certificate is using non
SHA-1 SKID (see rfc7093) or just 'unique number' (see rfc5280 4.2.1.2).
As a last resort user may specify arbitrary keyid using the new option.
Third option is to read keyid from the cert appended to the key file.

These commits create backward compatible ABI change for libimaevm,
because of adding additional parameter to imaevm_params - older
libimaevm can work with newer client.

Changes from v1:
- Extract keyid from cert associated to key file.
- Use sizeof instead of constant.

Changes since rfc version:
- `imaevm_params.keyid' now stored as native integer (instead of network
  order). Suggested by Stefan Berger.
  - Added support for `--keyid=@filename'.

Vitaly Chikunov (3):
  ima-evm-utils: Allow manual setting keyid for signing
  ima-evm-utils: Allow manual setting keyid from a cert file
  ima-evm-utils: Read keyid from the cert appended to the key file

 README                 |  4 +++
 src/evmctl.c           | 86 ++++++++++++++++++++++++++++++++++++++++++++++++++
 src/imaevm.h           |  1 +
 src/libimaevm.c        | 52 ++++++++++++++++++++++++++++--
 tests/sign_verify.test |  2 ++
 5 files changed, 142 insertions(+), 3 deletions(-)