Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1621243 > unrolled thread

[PATCH] KEYS: fix build failure with CONFIG_KEY_DH_OPERATIONS

Back to article view | Back to linux.kernel

Contents

  [PATCH] KEYS: fix build failure with CONFIG_KEY_DH_OPERATIONS Tobias Regnery <tobias.regnery@gmail.com> - 2017-04-11 14:50 +0200
    Re: [PATCH] KEYS: fix build failure with CONFIG_KEY_DH_OPERATIONS Stephan Müller <smueller@chronox.de> - 2017-04-11 15:50 +0200

#1621243 — [PATCH] KEYS: fix build failure with CONFIG_KEY_DH_OPERATIONS

With CONFIG_KEY_DH_OPERATIONS=y and CONFIG_CRYPTO=m we get the following
link error:

security/built-in.o: In function `__keyctl_dh_compute':
device_cgroup.c:(.text+0x925c): undefined reference to 'crypto_alloc_shash'
device_cgroup.c:(.text+0x9510): undefined reference to 'crypto_shash_update'
device_cgroup.c:(.text+0x9548): undefined reference to 'crypto_shash_update'
device_cgroup.c:(.text+0x957c): undefined reference to 'crypto_shash_final'
device_cgroup.c:(.text+0x95d0): undefined reference to 'crypto_shash_final'
device_cgroup.c:(.text+0x9710): undefined reference to 'crypto_destroy_tfm'

Fix this by selecting the crypto core in order to reach the functions above.

Fixes: f1c316a3ab9d ("KEYS: add SP800-56A KDF support for DH")
Signed-off-by: Tobias Regnery <tobias.regnery@gmail.com>
---
 security/keys/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/security/keys/Kconfig b/security/keys/Kconfig
index 4ac1b83a23f8..6fd95f76bfae 100644
--- a/security/keys/Kconfig
+++ b/security/keys/Kconfig
@@ -90,6 +90,7 @@ config KEY_DH_OPERATIONS
        bool "Diffie-Hellman operations on retained keys"
        depends on KEYS
        select MPILIB
+       select CRYPTO
        select CRYPTO_HASH
        help
 	 This option provides support for calculating Diffie-Hellman
-- 
2.11.0

[toc] | [next] | [standalone]

#1621287

Am Dienstag, 11. April 2017, 14:45:12 CEST schrieb Tobias Regnery:

Hi Tobias,

> With CONFIG_KEY_DH_OPERATIONS=y and CONFIG_CRYPTO=m we get the following
> link error:

Thanks for the patch. I just released the very same patch :-)

David, I guess it makes more sense now to simply apply that patch instead of 
me going back and update the original patch.

Thanks.

Ciao
Stephan

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web