Horton

Thread

Commits

  1. Use OpenSSL-specific ifdefs in sha2.h

  1. Backend specific ifdefs in sha2.h

    Daniel Gustafsson <daniel@yesql.se> — 2019-06-13T07:32:28Z

    The #ifdef guards in sha2.h are using USE_SSL when they in fact are guarding
the inclusion of OpenSSL specific code.  This has never caused any issues as
there only is a single supported TLS backend in core so far, but since we’ve
spent a significant amount of energy on making the TLS backend non-hardcoded
it seems we should fix this too.  The Makefile around sha2.c/sha2_openssl.c is
already testing for openssl rather than ssl (which given src/Makefile.global
variables makes perfect sense of course).

cheers ./daniel

  2. Re: Backend specific ifdefs in sha2.h

    Michael Paquier <michael@paquier.xyz> — 2019-06-13T08:29:27Z

    On Thu, Jun 13, 2019 at 09:32:28AM +0200, Daniel Gustafsson wrote:
> The #ifdef guards in sha2.h are using USE_SSL when they in fact are guarding
> the inclusion of OpenSSL specific code.  This has never caused any issues as
> there only is a single supported TLS backend in core so far, but since we’ve
> spent a significant amount of energy on making the TLS backend non-hardcoded
> it seems we should fix this too.  The Makefile around sha2.c/sha2_openssl.c is
> already testing for openssl rather than ssl (which given src/Makefile.global
> variables makes perfect sense of course).

Right, good catch.  I would not back-patch that though as currently
USE_SSL <=> USE_OPENSSL.  Any suggestions or thoughts from others?
--
Michael

  3. Re: Backend specific ifdefs in sha2.h

    Daniel Gustafsson <daniel@yesql.se> — 2019-06-13T08:31:23Z

    > On 13 Jun 2019, at 10:29, Michael Paquier <michael@paquier.xyz> wrote:

> I would not back-patch that though as currently
> USE_SSL <=> USE_OPENSSL.

Right, there is no use in backporting of course.

cheers ./daniel

  4. Re: Backend specific ifdefs in sha2.h

    Michael Paquier <michael@paquier.xyz> — 2019-06-14T00:10:55Z

    On Thu, Jun 13, 2019 at 10:31:23AM +0200, Daniel Gustafsson wrote:
> Right, there is no use in backporting of course.

And applied now, in time for beta2.
--
Michael