[openssl/openssl] 8225d8: use evp_kem_free in evp_kem_from_algorithm

  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 8225d82f04cf49412c5779ee19c3d5d812e1f265
      https://github.com/openssl/openssl/commit/8225d82f04cf49412c5779ee19c3d5d812e1f265
  Author: Neil Horman <[email protected]>
  Date:   2026-07-01 (Wed, 01 Jul 2026)

  Changed paths:
    M crypto/evp/kem.c

  Log Message:
  -----------
  use evp_kem_free in evp_kem_from_algorithm

evp_kem_from_algorithm, in its error path frees the allocated kem
with EVP_KEM_free, but thats a no-op now, and we actually want to
free it to avoid leaks, so we should use evp_kem_free (the internal
function that acutally does free the alg) instead.

Fixes https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1695456

Reviewed-by: Saša Nedvědický <[email protected]>
Reviewed-by: Eugene Syromiatnikov <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
MergeDate: Wed Jul  1 15:32:16 2026
(Merged from https://github.com/openssl/openssl/pull/31748)


  Commit: e4551d81c3239c31fd2c60ddb5926cca56c73a6d
      https://github.com/openssl/openssl/commit/e4551d81c3239c31fd2c60ddb5926cca56c73a6d
  Author: Neil Horman <[email protected]>
  Date:   2026-07-01 (Wed, 01 Jul 2026)

  Changed paths:
    M crypto/evp/exchange.c

  Log Message:
  -----------
  use evp_keyexch_free in evp_keyexch_from_algorithm

evp_keyexch_from_algorithm, in its error path frees the allocated kem
with EVP_KEM_free, but thats a no-op now, and we actually want to
free it to avoid leaks, so we should use evp_keyexch_free (the internal
function that acutally does free the alg) instead.

Fixes https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1695455

Reviewed-by: Saša Nedvědický <[email protected]>
Reviewed-by: Eugene Syromiatnikov <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
MergeDate: Wed Jul  1 15:32:18 2026
(Merged from https://github.com/openssl/openssl/pull/31748)


  Commit: 4c48ed73190fe282feb3858bdc48dd5fa8d03b96
      https://github.com/openssl/openssl/commit/4c48ed73190fe282feb3858bdc48dd5fa8d03b96
  Author: Neil Horman <[email protected]>
  Date:   2026-07-01 (Wed, 01 Jul 2026)

  Changed paths:
    M crypto/evp/keymgmt_meth.c

  Log Message:
  -----------
  use evp_keymgmt_free in keymgmt_from_algorithm

keymgmt_from_algorithm, in its error path frees the allocated keymgmt
with EVP_KEYMGMT_free, but thats a no-op now, and we actually want to
free it to avoid leaks, so we should use evp_keymgmt_free (the internal
function that acutally does free the alg) instead.

Fixes https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1695452

Reviewed-by: Saša Nedvědický <[email protected]>
Reviewed-by: Eugene Syromiatnikov <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
MergeDate: Wed Jul  1 15:32:21 2026
(Merged from https://github.com/openssl/openssl/pull/31748)


  Commit: 4d50403e5125d7d8190e6adff3d86727ba05c912
      https://github.com/openssl/openssl/commit/4d50403e5125d7d8190e6adff3d86727ba05c912
  Author: Neil Horman <[email protected]>
  Date:   2026-07-01 (Wed, 01 Jul 2026)

  Changed paths:
    M crypto/evp/signature.c

  Log Message:
  -----------
  use evp_signature_free in evp_signature_from_algorithm

evp_signature_from_algorithm, in its error path frees the allocated sig
with EVP_SIGNATURE_free, but thats a no-op now, and we actually want to
free it to avoid leaks, so we should use evp_signature_free (the internal
function that acutally does free the alg) instead.

Fixes https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1695450

Reviewed-by: Saša Nedvědický <[email protected]>
Reviewed-by: Eugene Syromiatnikov <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
MergeDate: Wed Jul  1 15:32:23 2026
(Merged from https://github.com/openssl/openssl/pull/31748)


  Commit: 8a072a3dabe4df116460f8910735569df321b97b
      https://github.com/openssl/openssl/commit/8a072a3dabe4df116460f8910735569df321b97b
  Author: Neil Horman <[email protected]>
  Date:   2026-07-01 (Wed, 01 Jul 2026)

  Changed paths:
    M crypto/evp/asymcipher.c

  Log Message:
  -----------
  use evp_asym_cipher_free in evp_asym_cipher_from_algorithm

evp_asym_cipher_from_algorithm, in its error path frees the allocated
cipher with EVP_ASM_CIPHER_free, but thats a no-op now, and we actually want to
free it to avoid leaks, so we should use evp_asym_cipher_free (the internal
function that acutally does free the alg) instead.

Fixes https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1695449

Reviewed-by: Saša Nedvědický <[email protected]>
Reviewed-by: Eugene Syromiatnikov <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
MergeDate: Wed Jul  1 15:32:25 2026
(Merged from https://github.com/openssl/openssl/pull/31748)


Compare: https://github.com/openssl/openssl/compare/5d61a36a9425...8a072a3dabe4

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

-- 
You received this message because you are subscribed to the Google Groups "openssl-commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
To view this discussion visit https://groups.google.com/a/openssl.org/d/msgid/openssl-commits/openssl/openssl/push/refs/heads/master/5d61a3-8a072a%40github.com.