linux-security-module.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ima: Remove deprecated IMA_TRUSTED_KEYRING Kconfig
@ 2023-07-11 16:44 Nayna Jain
  2023-07-27 17:34 ` Mimi Zohar
  2023-09-19  7:32 ` Oleksandr Tymoshenko
  0 siblings, 2 replies; 4+ messages in thread
From: Nayna Jain @ 2023-07-11 16:44 UTC (permalink / raw)
  To: linux-integrity @ vger . kernel . org
  Cc: Mimi Zohar, Linux Kernel Mailing List, linux-security-module,
	Nayna Jain

Time to remove "IMA_TRUSTED_KEYRING".

Fixes: f4dc37785e9b ("integrity: define '.evm' as a builtin 'trusted' keyring") # v4.5+
Signed-off-by: Nayna Jain <nayna@linux.ibm.com>
---
 security/integrity/ima/Kconfig | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/security/integrity/ima/Kconfig b/security/integrity/ima/Kconfig
index 60a511c6b583..c17660bf5f34 100644
--- a/security/integrity/ima/Kconfig
+++ b/security/integrity/ima/Kconfig
@@ -248,18 +248,6 @@ config IMA_APPRAISE_MODSIG
 	   The modsig keyword can be used in the IMA policy to allow a hook
 	   to accept such signatures.
 
-config IMA_TRUSTED_KEYRING
-	bool "Require all keys on the .ima keyring be signed (deprecated)"
-	depends on IMA_APPRAISE && SYSTEM_TRUSTED_KEYRING
-	depends on INTEGRITY_ASYMMETRIC_KEYS
-	select INTEGRITY_TRUSTED_KEYRING
-	default y
-	help
-	   This option requires that all keys added to the .ima
-	   keyring be signed by a key on the system trusted keyring.
-
-	   This option is deprecated in favor of INTEGRITY_TRUSTED_KEYRING
-
 config IMA_KEYRINGS_PERMIT_SIGNED_BY_BUILTIN_OR_SECONDARY
 	bool "Permit keys validly signed by a built-in or secondary CA cert (EXPERIMENTAL)"
 	depends on SYSTEM_TRUSTED_KEYRING
-- 
2.31.1


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

* Re: [PATCH] ima: Remove deprecated IMA_TRUSTED_KEYRING Kconfig
  2023-07-11 16:44 [PATCH] ima: Remove deprecated IMA_TRUSTED_KEYRING Kconfig Nayna Jain
@ 2023-07-27 17:34 ` Mimi Zohar
  2023-09-19  7:32 ` Oleksandr Tymoshenko
  1 sibling, 0 replies; 4+ messages in thread
From: Mimi Zohar @ 2023-07-27 17:34 UTC (permalink / raw)
  To: Nayna Jain, linux-integrity @ vger . kernel . org
  Cc: Linux Kernel Mailing List, linux-security-module

On Tue, 2023-07-11 at 12:44 -0400, Nayna Jain wrote:
> Time to remove "IMA_TRUSTED_KEYRING".
> 
> Fixes: f4dc37785e9b ("integrity: define '.evm' as a builtin 'trusted' keyring") # v4.5+
> Signed-off-by: Nayna Jain <nayna@linux.ibm.com>

Thanks, Nayna.   The patch is now queued in next-integrity-testing.

-- 
thanks,

Mimi


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

* [PATCH] ima: Remove deprecated IMA_TRUSTED_KEYRING Kconfig
  2023-07-11 16:44 [PATCH] ima: Remove deprecated IMA_TRUSTED_KEYRING Kconfig Nayna Jain
  2023-07-27 17:34 ` Mimi Zohar
@ 2023-09-19  7:32 ` Oleksandr Tymoshenko
  2023-09-20  2:29   ` Nayna
  1 sibling, 1 reply; 4+ messages in thread
From: Oleksandr Tymoshenko @ 2023-09-19  7:32 UTC (permalink / raw)
  To: nayna; +Cc: linux-integrity, linux-kernel, linux-security-module, zohar

Hello,

There are two Kconfigs that depend on IMA_TRUSTED_KEYRING:
IMA_LOAD_X509 and IMA_BLACKLIST_KEYRING. Removing IMA_TRUSTED_KEYRING
makes them unreachable. Should they be removed too or should
the dependency clauses be removed?



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

* Re: [PATCH] ima: Remove deprecated IMA_TRUSTED_KEYRING Kconfig
  2023-09-19  7:32 ` Oleksandr Tymoshenko
@ 2023-09-20  2:29   ` Nayna
  0 siblings, 0 replies; 4+ messages in thread
From: Nayna @ 2023-09-20  2:29 UTC (permalink / raw)
  To: Oleksandr Tymoshenko, nayna
  Cc: linux-integrity, linux-kernel, linux-security-module, zohar


On 9/19/23 03:32, Oleksandr Tymoshenko wrote:
> Hello,
>
> There are two Kconfigs that depend on IMA_TRUSTED_KEYRING:
> IMA_LOAD_X509 and IMA_BLACKLIST_KEYRING. Removing IMA_TRUSTED_KEYRING
> makes them unreachable. Should they be removed too or should
> the dependency clauses be removed?
>
>
Thanks Oleksandr for noticing this. Since IMA_TRUSTED_KEYRING is 
deprecated in favor of INTEGRITY_TRUSTED_KEYRING, I think the dependency 
clause should be updated to use INTEGRITY_TRUSTED_KEYRING.

Thanks & Regards,

    - Nayna


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

end of thread, other threads:[~2023-09-20  2:29 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-11 16:44 [PATCH] ima: Remove deprecated IMA_TRUSTED_KEYRING Kconfig Nayna Jain
2023-07-27 17:34 ` Mimi Zohar
2023-09-19  7:32 ` Oleksandr Tymoshenko
2023-09-20  2:29   ` Nayna

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).