diff mbox series

crypto: ecrdsa - select ASN1 and OID_REGISTRY for EC-RDSA

Message ID 20190423162954.7859-1-vt@altlinux.org (mailing list archive)
State Superseded
Delegated to: Herbert Xu
Headers show
Series crypto: ecrdsa - select ASN1 and OID_REGISTRY for EC-RDSA | expand

Commit Message

Vitaly Chikunov April 23, 2019, 4:29 p.m. UTC
Fix undefined symbol issue in ecrdsa_generic module when ASN1
or OID_REGISTRY aren't enabled in the config by selecting these
options for CRYPTO_ECRDSA.

ERROR: "asn1_ber_decoder" [crypto/ecrdsa_generic.ko] undefined!
ERROR: "look_up_OID" [crypto/ecrdsa_generic.ko] undefined!

Reported-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Vitaly Chikunov <vt@altlinux.org>
---
 crypto/Kconfig | 2 ++
 1 file changed, 2 insertions(+)

Comments

Randy Dunlap April 23, 2019, 8:20 p.m. UTC | #1
On 4/23/19 9:29 AM, Vitaly Chikunov wrote:
> Fix undefined symbol issue in ecrdsa_generic module when ASN1
> or OID_REGISTRY aren't enabled in the config by selecting these
> options for CRYPTO_ECRDSA.
> 
> ERROR: "asn1_ber_decoder" [crypto/ecrdsa_generic.ko] undefined!
> ERROR: "look_up_OID" [crypto/ecrdsa_generic.ko] undefined!
> 
> Reported-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: Vitaly Chikunov <vt@altlinux.org>

Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested

Thanks.

My tools would prefer to see the diff at one tree level higher:


> ---
>  crypto/Kconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git crypto/Kconfig crypto/Kconfig

  diff --git a/crypto/Kconfig b/crypto/Kconfig

for any spelling of a and b.


> index 4446833f6eca..3d056e7da65f 100644
> --- crypto/Kconfig
> +++ crypto/Kconfig

  --- a/crypto/Kconfig
  +++ b/crypto/Kconfig

> @@ -264,6 +264,8 @@ config CRYPTO_ECRDSA
>  	select CRYPTO_ECC
>  	select CRYPTO_AKCIPHER
>  	select CRYPTO_STREEBOG
> +	select OID_REGISTRY
> +	select ASN1
>  	help
>  	  Elliptic Curve Russian Digital Signature Algorithm (GOST R 34.10-2012,
>  	  RFC 7091, ISO/IEC 14888-3:2018) is one of the Russian cryptographic
>
Herbert Xu April 25, 2019, 7:52 a.m. UTC | #2
On Tue, Apr 23, 2019 at 07:29:54PM +0300, Vitaly Chikunov wrote:
> Fix undefined symbol issue in ecrdsa_generic module when ASN1
> or OID_REGISTRY aren't enabled in the config by selecting these
> options for CRYPTO_ECRDSA.
> 
> ERROR: "asn1_ber_decoder" [crypto/ecrdsa_generic.ko] undefined!
> ERROR: "look_up_OID" [crypto/ecrdsa_generic.ko] undefined!
> 
> Reported-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: Vitaly Chikunov <vt@altlinux.org>
> ---
>  crypto/Kconfig | 2 ++
>  1 file changed, 2 insertions(+)

Patch applied.  Thanks.
diff mbox series

Patch

diff --git crypto/Kconfig crypto/Kconfig
index 4446833f6eca..3d056e7da65f 100644
--- crypto/Kconfig
+++ crypto/Kconfig
@@ -264,6 +264,8 @@  config CRYPTO_ECRDSA
 	select CRYPTO_ECC
 	select CRYPTO_AKCIPHER
 	select CRYPTO_STREEBOG
+	select OID_REGISTRY
+	select ASN1
 	help
 	  Elliptic Curve Russian Digital Signature Algorithm (GOST R 34.10-2012,
 	  RFC 7091, ISO/IEC 14888-3:2018) is one of the Russian cryptographic