[openssl/openssl] 2e5f29: Fix mem leak in threadpool_test.c

Hongren Zheng noreply at github.com
Wed May 22 11:28:26 UTC 2024


  Branch: refs/heads/openssl-3.2
  Home:   https://github.com/openssl/openssl
  Commit: 2e5f297b71bbd9037d2c63b8967b2e6f43ef43cb
      https://github.com/openssl/openssl/commit/2e5f297b71bbd9037d2c63b8967b2e6f43ef43cb
  Author: shridhar kalavagunta <coolshrid at hotmail.com>
  Date:   2024-05-20 (Mon, 20 May 2024)

  Changed paths:
    M test/threadpool_test.c

  Log Message:
  -----------
  Fix mem leak in threadpool_test.c

Fixes #24104

Added a goto label for cleanup.

Reviewed-by: Paul Dale <ppzgs1 at gmail.com>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24412)

(cherry picked from commit 4dbd4925dfc61d93df678df607504f62b0ac3dcc)


  Commit: 6002db89a69df55ccba6e43af12b476ca7950665
      https://github.com/openssl/openssl/commit/6002db89a69df55ccba6e43af12b476ca7950665
  Author: Hongren Zheng <i at zenithal.me>
  Date:   2024-05-20 (Mon, 20 May 2024)

  Changed paths:
    M providers/fips/fipsprov.c

  Log Message:
  -----------
  fips provider: explicitly setup cpuid when initializing

Fixes: #23979

Previously fips module relied on OPENSSL_cpuid_setup
being used as constructor by the linker to correctly
setup the capability vector, either via .section .init
(for x86_64) or via __attribute__((constructor)).

This would make ld.so call OPENSSL_cpuid_setup before
the init function for fips module. However, this early
constructing behavior has several disadvantages:

1. Not all platform/toolchain supports such behavior

2. Initialisation sequence is not well defined, and
some function might not be initialized when cpuid_setup
is called

3. Implicit path is hard to maintain and debug

Reviewed-by: Paul Dale <ppzgs1 at gmail.com>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24419)

(cherry picked from commit a192b2439c0207ce1b04ba6137329b68f9e23680)


Compare: https://github.com/openssl/openssl/compare/aa22a4d6f30a...6002db89a69d

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