[openssl/openssl] 1316aa: Add check for public key presence on sm2 signing

Neil Horman noreply at github.com
Thu May 2 15:46:02 UTC 2024


  Branch: refs/heads/openssl-3.1
  Home:   https://github.com/openssl/openssl
  Commit: 1316aa05aae57cf47d8c8bfca38aaa042db1518f
      https://github.com/openssl/openssl/commit/1316aa05aae57cf47d8c8bfca38aaa042db1518f
  Author: Neil Horman <nhorman at openssl.org>
  Date:   2024-05-02 (Thu, 02 May 2024)

  Changed paths:
    M crypto/sm2/sm2_sign.c
    M test/sm2_internal_test.c

  Log Message:
  -----------
  Add check for public key presence on sm2 signing

SM2 requires that the public EC_POINT be present in a key when signing.
If its not there we crash on a NULL pointer.  Add a check to ensure that
its present, and raise an error if its not

Reviewed-by: Paul Yang <kaishen.yy at antfin.com>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Matt Caswell <matt at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24078)


  Commit: 54673b93594a71c9f8052a1df1a7c6bf07c49f4d
      https://github.com/openssl/openssl/commit/54673b93594a71c9f8052a1df1a7c6bf07c49f4d
  Author: Neil Horman <nhorman at openssl.org>
  Date:   2024-05-02 (Thu, 02 May 2024)

  Changed paths:
    M doc/man7/EVP_PKEY-SM2.pod

  Log Message:
  -----------
  Add docs noting requirements for SM2 signing

Reviewed-by: Paul Yang <kaishen.yy at antfin.com>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Matt Caswell <matt at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24078)


Compare: https://github.com/openssl/openssl/compare/e4a87616c07a...54673b93594a

To unsubscribe from these emails, change your notification settings at https://github.com/openssl/openssl/settings/notifications


More information about the openssl-commits mailing list