Re: [PATCH v8 0/3] integrity: support including firmware ".platform" keys at build time

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 2/20/22 18:02, Jarkko Sakkinen wrote:
On Tue, Jan 11, 2022 at 01:36:44PM -0500, Nayna Jain wrote:
Some firmware support secure boot by embedding static keys to verify the
Linux kernel during boot. However, these firmware do not expose an
interface for the kernel to load firmware keys onto the ".platform"
keyring, preventing the kernel from verifying the kexec kernel image
signature.

This patchset exports load_certificate_list() and defines a new function
load_builtin_platform_cert() to load compiled in certificates onto the
".platform" keyring.

Changelog:

v8:
* Includes Jarkko's feedback on patch description and removed Reported-by
for Patch 1.

v7:
* Incldues Jarkko's feedback on patch description for Patch 1 and 3.

v6:
* Includes Jarkko's feedback:
  * Split Patch 2 into two.
  * Update Patch description.

v5:
* Renamed load_builtin_platform_cert() to load_platform_certificate_list()
and config INTEGRITY_PLATFORM_BUILTIN_KEYS to INTEGRITY_PLATFORM_KEYS, as
suggested by Mimi Zohar.

v4:
* Split into two patches as per Mimi Zohar and Dimitri John Ledkov
recommendation.

v3:
* Included Jarkko's feedback
  ** updated patch description to include approach.
  ** removed extern for function declaration in the .h file.
* Included load_certificate_list() within #ifdef CONFIG_KEYS condition.

v2:
* Fixed the error reported by kernel test robot
* Updated patch description based on Jarkko's feedback.

Nayna Jain (3):
   certs: export load_certificate_list() to be used outside certs/
   integrity: make integrity_keyring_from_id() non-static
   integrity: support including firmware ".platform" keys at build time

  certs/Makefile                                |  5 ++--
  certs/blacklist.c                             |  1 -
  certs/common.c                                |  2 +-
  certs/common.h                                |  9 -------
  certs/system_keyring.c                        |  1 -
  include/keys/system_keyring.h                 |  6 +++++
  security/integrity/Kconfig                    | 10 +++++++
  security/integrity/Makefile                   | 17 +++++++++++-
  security/integrity/digsig.c                   |  2 +-
  security/integrity/integrity.h                |  6 +++++
  .../integrity/platform_certs/platform_cert.S  | 23 ++++++++++++++++
  .../platform_certs/platform_keyring.c         | 26 +++++++++++++++++++
  12 files changed, 92 insertions(+), 16 deletions(-)
  delete mode 100644 certs/common.h
  create mode 100644 security/integrity/platform_certs/platform_cert.S

--
2.27.0
To sort out tree conflicts: what if I pick these patches? They look fine
to me now. I can try to fix the possible merge conflicts and you can check
them before I make a PR.

Sounds good. Thanks !!

Thanks & Regards,

     - Nayna




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux Kernel]     [Linux Kernel Hardening]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux