[openssl/openssl] 538f9d: After initializing a provider, check if its output...

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


  Branch: refs/heads/openssl-3.1
  Home:   https://github.com/openssl/openssl
  Commit: 538f9d2d2845fb5332cc3c1021bd70b92611aeb6
      https://github.com/openssl/openssl/commit/538f9d2d2845fb5332cc3c1021bd70b92611aeb6
  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: 710509485f654f033fde483445496fb14b755fa3
      https://github.com/openssl/openssl/commit/710509485f654f033fde483445496fb14b755fa3
  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: 24fe507c96415e3fac31ed1eb89b7632d649f552
      https://github.com/openssl/openssl/commit/24fe507c96415e3fac31ed1eb89b7632d649f552
  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/3af29bf9f99d...24fe507c9641


More information about the openssl-commits mailing list