Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 14 Dec 2018 13:39:03 +1100
From:      Kubilay Kocak <koobs@FreeBSD.org>
To:        Jung-uk Kim <jkim@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r342057 - head/crypto/openssl/crypto/engine
Message-ID:  <216fb35b-28d1-faaa-d115-1f4a601212f0@FreeBSD.org>
In-Reply-To: <201812140106.wBE16Zgr058499@repo.freebsd.org>
References:  <201812140106.wBE16Zgr058499@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 14/12/2018 12:06 pm, Jung-uk Kim wrote:
> Author: jkim
> Date: Fri Dec 14 01:06:34 2018
> New Revision: 342057
> URL: https://svnweb.freebsd.org/changeset/base/342057
> 
> Log:
>    Do not complain when /dev/crypto does not exist.
>    
>    Now the new devcrypto engine is enabled since r342009, many users started
>    seeing "Could not open /dev/crypto: No such file or directory".  Disable
>    the annoying error message as it is not very useful anyway.
>    
>    Note the patch was submitted upstream.
>    
>    https://github.com/openssl/openssl/pull/7896
> 
> Modified:
>    head/crypto/openssl/crypto/engine/eng_devcrypto.c
> 
> Modified: head/crypto/openssl/crypto/engine/eng_devcrypto.c
> ==============================================================================
> --- head/crypto/openssl/crypto/engine/eng_devcrypto.c	Fri Dec 14 00:40:38 2018	(r342056)
> +++ head/crypto/openssl/crypto/engine/eng_devcrypto.c	Fri Dec 14 01:06:34 2018	(r342057)
> @@ -24,6 +24,8 @@
>   
>   #include "internal/engine.h"
>   
> +/* #define ENGINE_DEVCRYPTO_DEBUG */
> +
>   #ifdef CRYPTO_ALGORITHM_MIN
>   # define CHECK_BSD_STYLE_MACROS
>   #endif
> @@ -615,6 +617,9 @@ void engine_load_devcrypto_int()
>       ENGINE *e = NULL;
>   
>       if ((cfd = open("/dev/crypto", O_RDWR, 0)) < 0) {
> +#ifndef ENGINE_DEVCRYPTO_DEBUG
> +        if (errno != ENOENT)
> +#endif
>           fprintf(stderr, "Could not open /dev/crypto: %s\n", strerror(errno));
>           return;
>       }

How trivially could devcrypto_debug  be modified to be a runtime 
configuration (say sysctl) setting?



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?216fb35b-28d1-faaa-d115-1f4a601212f0>