[openssl/openssl] f0c3bb: ossl_param_build_set_multi_key_bn(): Do not set NU...

[Neil Horman]

  Branch: refs/heads/openssl-3.1
  Home:   https://github.com/openssl/openssl
  Commit: f0c3bb1ac11fb25659f0b88a769d8a10edf82355
      https://github.com/openssl/openssl/commit/f0c3bb1ac11fb25659f0b88a769d8a10edf82355
  Author: Tomas Mraz <tomas at openssl.org>
  Date:   2023-10-18 (Wed, 18 Oct 2023)

  Changed paths:
    M crypto/param_build_set.c

  Log Message:
  -----------
  ossl_param_build_set_multi_key_bn(): Do not set NULL BIGNUMs

This makes them zeroes otherwise
where NULLs actually mean the values aren't present.

Fixes #21935

Reviewed-by: Richard Levitte <levitte at openssl.org>
Reviewed-by: Tom Cosgrove <tom.cosgrove at arm.com>
(Merged from https://github.com/openssl/openssl/pull/22334)

(cherry picked from commit 15a39e7025e0ed4e31664c499894006e41582068)


  Commit: 37332807a0bfcb9aca8e70d2ebb72cdb81ab379a
      https://github.com/openssl/openssl/commit/37332807a0bfcb9aca8e70d2ebb72cdb81ab379a
  Author: Neil Horman <nhorman at openssl.org>
  Date:   2023-10-18 (Wed, 18 Oct 2023)

  Changed paths:
    M crypto/rsa/rsa_backend.c

  Log Message:
  -----------
  remove sanity check from ossl_rsa_todata

Theres no reason we should gate ossl_rsa_todata on there being a minimum
set of parameters. EVP_PKEY_todata makes no guarantees about the
validity of a key, it only returns the parameters that are set in the
requested key, whatever they may be.  Remove the check.

Fixes #21935

Reviewed-by: Richard Levitte <levitte at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22334)

(cherry picked from commit 4ad3a44ba45a4026170336161228d435f6784564)


  Commit: dd3ad58faab9d7495efdb0c45c516cf1ba18d650
      https://github.com/openssl/openssl/commit/dd3ad58faab9d7495efdb0c45c516cf1ba18d650
  Author: Neil Horman <nhorman at openssl.org>
  Date:   2023-10-18 (Wed, 18 Oct 2023)

  Changed paths:
    M crypto/rsa/rsa_lib.c

  Log Message:
  -----------
  Dont require CRT params on ossl_rsa_set0_all_params

Its not required that crt params be available in an RSA key, so don't
perform an error check on them

Fixes #29135

Reviewed-by: Richard Levitte <levitte at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22334)

(cherry picked from commit 2647726bd3ca63dc5f07ae3f10e16dff35d95626)


  Commit: 52f96edf9c363f1ce977fd343b1a512985e0bec9
      https://github.com/openssl/openssl/commit/52f96edf9c363f1ce977fd343b1a512985e0bec9
  Author: Neil Horman <nhorman at openssl.org>
  Date:   2023-10-18 (Wed, 18 Oct 2023)

  Changed paths:
    M test/rsa_test.c

  Log Message:
  -----------
  Add a test to confirm that legacy rsa keys work

Reviewed-by: Richard Levitte <levitte at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22334)

(cherry picked from commit e62097f48c3d0b8b61ca6a061b8098b0086b3fbc)


Compare: https://github.com/openssl/openssl/compare/339bbcd70c1d...52f96edf9c36