[openssl/openssl] 506ff2: Make the activate setting more intuitive

Neil Horman noreply at github.com
Thu Dec 21 14:39:14 UTC 2023


  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 506ff20662a228b17840f0b49865a927a45c2908
      https://github.com/openssl/openssl/commit/506ff20662a228b17840f0b49865a927a45c2908
  Author: Neil Horman <nhorman at openssl.org>
  Date:   2023-12-21 (Thu, 21 Dec 2023)

  Changed paths:
    M CHANGES.md
    M crypto/provider_conf.c
    M doc/man5/config.pod
    M test/default-and-fips.cnf
    M test/default.cnf
    M test/evp_fetch_prov_test.c

  Log Message:
  -----------
  Make the activate setting more intuitive

Currently, a provider is activated from our config file using the
activate parameter.  However, the presence of the config parameter is
sufficient to trigger activation, leading to a counterintuitive
situation in which setting "activate = 0" still activates the provider

Make activation more intuitive by requiring that activate be set to one
of yes|true|1 to trigger activation.  Any other value, as well as
omitting the parameter entirely, prevents activation (and also maintains
backward compatibility.

It seems a bit heavyweight to create a test specifically to validate the
plurality of these settings.  Instead, modify the exiting openssl config
files in the test directory to use variants of these settings, and
augment the default.cnf file to include a provider section that is
explicitly disabled

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Matthias St. Pierre <Matthias.St.Pierre at ncp-e.com>
Reviewed-by: Matt Caswell <matt at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22906)




More information about the openssl-commits mailing list