[openssl/openssl] 08d420: providers: Set the size of EC signature on s390.

openssl-machine noreply at github.com
Thu Aug 4 13:29:02 UTC 2022


  Branch: refs/heads/openssl-3.0
  Home:   https://github.com/openssl/openssl
  Commit: 08d420dd8edb9b6b5a0616171b30b5a53beb630b
      https://github.com/openssl/openssl/commit/08d420dd8edb9b6b5a0616171b30b5a53beb630b
  Author: Sebastian Andrzej Siewior <sebastian at breakpoint.cc>
  Date:   2022-08-04 (Thu, 04 Aug 2022)

  Changed paths:
    M providers/implementations/signature/eddsa_sig.c

  Log Message:
  -----------
  providers: Set the size of EC signature on s390.

The s390x provides its custom implementation for the creation of the
ed448 and ed25519 signatures. Unfortunately it does not set the size.
Users that rely of this return parameter end up with wrong values and
will compare wrong sizes of signature.

Set the proper size of the returned signature on success. Set an error
if the signing operation fails.

Fixes: #18912
Signed-off-by: Sebastian Andrzej Siewior <sebastian at breakpoint.cc>

Reviewed-by: Matt Caswell <matt at openssl.org>
Reviewed-by: Todd Short <todd.short at me.com>
(Merged from https://github.com/openssl/openssl/pull/18928)

(cherry picked from commit bbedc052973b1c2fab7d7fb891d02aea393ff579)


  Commit: b3f53a2f6fa6ed2ccab0423aadc753fbb3d70878
      https://github.com/openssl/openssl/commit/b3f53a2f6fa6ed2ccab0423aadc753fbb3d70878
  Author: Sebastian Andrzej Siewior <sebastian at breakpoint.cc>
  Date:   2022-08-04 (Thu, 04 Aug 2022)

  Changed paths:
    M test/evp_test.c

  Log Message:
  -----------
  test/evp_test: Test if EVP_DigestSign() set signature's length.

Increase the signature's length, that is passed to EVP_DigestSign(). The
implementation should set this parameter back to the actual length, that
has been written.

This (oneshot_digestsign_test_run()) fails for the ed25519/ ed448
implementation of s390 prio the fix. The change in digestsign_test_run()
follows the same pattern and is for collecting bonus points.

Signed-off-by: Sebastian Andrzej Siewior <sebastian at breakpoint.cc>

Reviewed-by: Matt Caswell <matt at openssl.org>
Reviewed-by: Todd Short <todd.short at me.com>
(Merged from https://github.com/openssl/openssl/pull/18928)

(cherry picked from commit fc5888ccb60f33b366972299f30b976c4dc12162)


Compare: https://github.com/openssl/openssl/compare/c69410a78666...b3f53a2f6fa6


More information about the openssl-commits mailing list