lkml.org 
[lkml]   [2021]   [Mar]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3 0/4] keys: Add EFI_CERT_X509_GUID support for dbx/mokx entries
On Tue, Mar 09, 2021 at 01:20:42PM +0000, David Howells wrote:
>
> Here's my take on v5 of Eric Snowberg's patches[1]:
>
> This series of patches adds support for EFI_CERT_X509_GUID entries [2]. It has
> been expanded to not only include dbx entries but also entries in the mokx.
> Additionally Eric included his patches to preload these certificate [3].
>
> The patches can be found on the following branch:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git/log/?h=keys-cve-2020-26541-branch
>
> Changes:
>
> ver #3:
> - Rolled in changes from Eric to fix conditional building issues[7].
>
> ver #2:
> - Rolled in a fix to the second patch to include certs/common.h in
> certs/common.c[6].
>
> ver #1:
> - I've modified the first patch in the series to fix a configuration
> problem[4][5], to move the added functions to a more logical place
> within thefile and to add kerneldoc comments.
>
> Link: https://lore.kernel.org/r/20210122181054.32635-1-eric.snowberg@oracle.com [1]
> Link: https://patchwork.kernel.org/project/linux-security-module/patch/20200916004927.64276-1-eric.snowberg@oracle.com/ [2]
> Link: https://lore.kernel.org/patchwork/cover/1315485/ [3]
> Link: https://lore.kernel.org/r/bc2c24e3-ed68-2521-0bf4-a1f6be4a895d@infradead.org/ [4]
> Link: https://lore.kernel.org/r/20210225125638.1841436-1-arnd@kernel.org/ [5]
> Link: https://lore.kernel.org/r/EDA280F9-F72D-4181-93C7-CDBE95976FF7@oracle.com/ [6]
> Link: https://lore.kernel.org/r/161428671215.677100.6372209948022011988.stgit@warthog.procyon.org.uk/ # v1 posting
> Link: https://lore.kernel.org/r/161433310139.902181.11787442834918634133.stgit@warthog.procyon.org.uk/ # v2 posting
> Link: https://lore.kernel.org/r/20210304175030.184131-1-eric.snowberg@oracle.com/ [7]
>
> David
> ---
> Eric Snowberg (4):
> certs: Add EFI_CERT_X509_GUID support for dbx entries
> certs: Move load_system_certificate_list to a common function
> certs: Add ability to preload revocation certs
> integrity: Load mokx variables into the blacklist keyring
>
>
> certs/Kconfig | 8 +++
> certs/Makefile | 21 ++++++-
> certs/blacklist.c | 21 +++++++
> certs/common.c | 57 +++++++++++++++++++
> certs/common.h | 9 +++
> certs/revocation_certificates.S | 21 +++++++
> certs/system_keyring.c | 49 +---------------
> scripts/Makefile | 1 +
> security/integrity/platform_certs/load_uefi.c | 20 ++++++-
> 9 files changed, 156 insertions(+), 51 deletions(-)
> create mode 100644 certs/common.c
> create mode 100644 certs/common.h
> create mode 100644 certs/revocation_certificates.S
>
>
>

Looks good to me.

/Jarkko

\
 
 \ /
  Last update: 2021-03-10 20:36    [W:0.047 / U:0.240 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site