Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 13 Dec 2018 20:29:50 -0700
From:      Kevin Bowling <kevin.bowling@kev009.com>
To:        koobs@freebsd.org
Cc:        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:  <CAK7dMtCtK2HiWQr=cFXwUrP4TxZQD5b_nnNLXyYyk4PANWPAFQ@mail.gmail.com>
In-Reply-To: <216fb35b-28d1-faaa-d115-1f4a601212f0@FreeBSD.org>
References:  <201812140106.wBE16Zgr058499@repo.freebsd.org> <216fb35b-28d1-faaa-d115-1f4a601212f0@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
But why, you can trivially see the open() call with truss or more advanced
tracers if you are debugging this

On Thu, Dec 13, 2018 at 6:39 PM Kubilay Kocak <koobs@freebsd.org> wrote:

> 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?
> _______________________________________________
> svn-src-head@freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-head
> To unsubscribe, send any mail to "svn-src-head-unsubscribe@freebsd.org"
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAK7dMtCtK2HiWQr=cFXwUrP4TxZQD5b_nnNLXyYyk4PANWPAFQ>