All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] certs: make system keyring depend on x509 parser
@ 2022-07-17 23:11 Adam Borowski
  2022-07-18  6:55 ` Thorsten Leemhuis
  0 siblings, 1 reply; 3+ messages in thread
From: Adam Borowski @ 2022-07-17 23:11 UTC (permalink / raw)
  To: David Howells, David Woodhouse, keyrings, linux-kernel; +Cc: Adam Borowski

This code requires x509_load_certificate_list() to be built-in.

Fixes: 60050ffe3d770dd1df5b641aa48f49d07a54bd84
Signed-off-by: Adam Borowski <kilobyte@angband.pl>
---
Otherwise, we get a build failure.

 certs/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/certs/Kconfig b/certs/Kconfig
index 476755703cf8..bf9b511573d7 100644
--- a/certs/Kconfig
+++ b/certs/Kconfig
@@ -43,6 +43,7 @@ config SYSTEM_TRUSTED_KEYRING
 	bool "Provide system-wide ring of trusted keys"
 	depends on KEYS
 	depends on ASYMMETRIC_KEY_TYPE
+	depends on X509_CERTIFICATE_PARSER
 	help
 	  Provide a system keyring to which trusted keys can be added.  Keys in
 	  the keyring are considered to be trusted.  Keys may be added at will
-- 
2.36.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] certs: make system keyring depend on x509 parser
  2022-07-17 23:11 [PATCH] certs: make system keyring depend on x509 parser Adam Borowski
@ 2022-07-18  6:55 ` Thorsten Leemhuis
  2022-07-18 13:50   ` Adam Borowski
  0 siblings, 1 reply; 3+ messages in thread
From: Thorsten Leemhuis @ 2022-07-18  6:55 UTC (permalink / raw)
  To: Adam Borowski, David Howells, David Woodhouse, keyrings, linux-kernel
  Cc: Steven Rostedt

Hi, this is your Linux kernel regression tracker.

Thx for looking into this.

On 18.07.22 01:11, Adam Borowski wrote:
> This code requires x509_load_certificate_list() to be built-in.
> 
> Fixes: 60050ffe3d770dd1df5b641aa48f49d07a54bd84

Just BTW: per docs.kernel.org/process/submitting-patches.html this
should be

Fixes: 60050ffe3d77 ("certs: Move load_certificate_list() to be with the
asymmetric keys code")

But I write for a different reason: Could you also please add "Link:"
and "Reported-by:" tags for earlier reports about this issue please:

Reported-by: kernel test robot <lkp@intel.com>
Link: https://lore.kernel.org/all/202206221515.DqpUuvbQ-lkp@intel.com/
Reported-by: Steven Rostedt <rostedt@goodmis.org>
Link:
https://lore.kernel.org/all/20220712104554.408dbf42@gandalf.local.home/

tia

> […]

Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat)

P.S.: As the Linux kernel's regression tracker I deal with a lot of
reports and sometimes miss something important when writing mails like
this. If that's the case here, don't hesitate to tell me in a public
reply, it's in everyone's interest to set the public record straight.

P.P.S.: Let me tell regzbot to monitor this thread:

#regzbot ^backmonitor:
https://lore.kernel.org/all/20220712104554.408dbf42@gandalf.local.home/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] certs: make system keyring depend on x509 parser
  2022-07-18  6:55 ` Thorsten Leemhuis
@ 2022-07-18 13:50   ` Adam Borowski
  0 siblings, 0 replies; 3+ messages in thread
From: Adam Borowski @ 2022-07-18 13:50 UTC (permalink / raw)
  To: David Howells, David Woodhouse, keyrings, linux-kernel,
	Steven Rostedt, Thorsten Leemhuis
  Cc: Adam Borowski, kernel test robot

This code requires x509_load_certificate_list() to be built-in.

Fixes: 60050ffe3d77 ("certs: Move load_certificate_list() to be with the
asymmetric keys code")
Reported-by: kernel test robot <lkp@intel.com>
Link: https://lore.kernel.org/all/202206221515.DqpUuvbQ-lkp@intel.com/
Reported-by: Steven Rostedt <rostedt@goodmis.org>
Link: https://lore.kernel.org/all/20220712104554.408dbf42@gandalf.local.home/
Signed-off-by: Adam Borowski <kilobyte@angband.pl>
---
 certs/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/certs/Kconfig b/certs/Kconfig
index 476755703cf8..bf9b511573d7 100644
--- a/certs/Kconfig
+++ b/certs/Kconfig
@@ -43,6 +43,7 @@ config SYSTEM_TRUSTED_KEYRING
 	bool "Provide system-wide ring of trusted keys"
 	depends on KEYS
 	depends on ASYMMETRIC_KEY_TYPE
+	depends on X509_CERTIFICATE_PARSER
 	help
 	  Provide a system keyring to which trusted keys can be added.  Keys in
 	  the keyring are considered to be trusted.  Keys may be added at will
-- 
2.36.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-07-18 13:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-17 23:11 [PATCH] certs: make system keyring depend on x509 parser Adam Borowski
2022-07-18  6:55 ` Thorsten Leemhuis
2022-07-18 13:50   ` Adam Borowski

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.