[openssl/openssl] 4266cd: After initializing a provider, check if its output...

Richard Levitte noreply at github.com
Mon Dec 4 14:17:50 UTC 2023


  Branch: refs/heads/openssl-3.2
  Home:   https://github.com/openssl/openssl
  Commit: 4266cd8d3db124d32f9cf0fb9a265da9bf24cb24
      https://github.com/openssl/openssl/commit/4266cd8d3db124d32f9cf0fb9a265da9bf24cb24
  Author: Richard Levitte <levitte at openssl.org>
  Date:   2023-12-04 (Mon, 04 Dec 2023)

  Changed paths:
    M crypto/provider_core.c

  Log Message:
  -----------
  After initializing a provider, check if its output dispatch table is NULL

If the provider's output dispatch table is NULL, trying to parse it causes a
crash.  Let's not do that.

Reviewed-by: Dmitry Belyavskiy <beldmit at gmail.com>
Reviewed-by: Todd Short <todd.short at me.com>
(Merged from https://github.com/openssl/openssl/pull/22866)

(cherry picked from commit 8fa65a6648554087a67102372e5e6c8b0fae0158)


  Commit: 94d838026d21bca7077c2246e4e795a2f108df8a
      https://github.com/openssl/openssl/commit/94d838026d21bca7077c2246e4e795a2f108df8a
  Author: Richard Levitte <levitte at openssl.org>
  Date:   2023-12-04 (Mon, 04 Dec 2023)

  Changed paths:
    M test/build.info
    A test/p_minimal.c
    M test/recipes/04-test_provider.t

  Log Message:
  -----------
  Add a minimal test provider

We test its validity by trying to load it.

Reviewed-by: Dmitry Belyavskiy <beldmit at gmail.com>
Reviewed-by: Todd Short <todd.short at me.com>
(Merged from https://github.com/openssl/openssl/pull/22866)

(cherry picked from commit 31c2c12f2dada75c334f6a9aa60c8424cf4fd040)


  Commit: e84eea81823344dd20bb8d60b0331626d11f9445
      https://github.com/openssl/openssl/commit/e84eea81823344dd20bb8d60b0331626d11f9445
  Author: Richard Levitte <levitte at openssl.org>
  Date:   2023-12-04 (Mon, 04 Dec 2023)

  Changed paths:
    M apps/list.c

  Log Message:
  -----------
  Make 'openssl list' less sensitive for providers without params

When a provider can't return parameters, make that a warning instead of an
error, and continue to list further providers.

Reviewed-by: Dmitry Belyavskiy <beldmit at gmail.com>
Reviewed-by: Todd Short <todd.short at me.com>
(Merged from https://github.com/openssl/openssl/pull/22866)

(cherry picked from commit 7ebaab7689f66ede1f960c42be3446922e3f5e21)


Compare: https://github.com/openssl/openssl/compare/cc493c030451...e84eea818233


More information about the openssl-commits mailing list