[openssl/openssl] 64963c: signature/sm2_sig.c: Add the check for the EVP_MD_...

[JiashengJiang]

  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 64963c8b7a11728b5d252420f56f82532c14076d
      https://github.com/openssl/openssl/commit/64963c8b7a11728b5d252420f56f82532c14076d
  Author: Jiasheng Jiang <jiasheng at purdue.edu>
  Date:   2024-03-30 (Sat, 30 Mar 2024)

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

  Log Message:
  -----------
  signature/sm2_sig.c: Add the check for the EVP_MD_CTX_get_size()

Add the check for the return value of EVP_MD_CTX_get_size() to avoid invalid negative numbers.

Fixes: d0b79f8631 ("Add SM2 signature algorithm to default provider")
Signed-off-by: Jiasheng Jiang <jiasheng at purdue.edu>

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Todd Short <todd.short at me.com>
Reviewed-by: Neil Horman <nhorman at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/23959)


  Commit: 2b6f307721db97d9bd7ca5ad4abf12b90ef581cd
      https://github.com/openssl/openssl/commit/2b6f307721db97d9bd7ca5ad4abf12b90ef581cd
  Author: Jiasheng Jiang <jiasheng at purdue.edu>
  Date:   2024-03-30 (Sat, 30 Mar 2024)

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

  Log Message:
  -----------
  Break the if statement up into 2 if statements

Break the if statement up into 2 if statements to avoid call
EVP_MD_get_size() twice.

Signed-off-by: Jiasheng Jiang <jiasheng at purdue.edu>

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Todd Short <todd.short at me.com>
Reviewed-by: Neil Horman <nhorman at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/23959)


Compare: https://github.com/openssl/openssl/compare/4169d58c8557...2b6f307721db

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