[openssl/openssl] f5f4bc: Fix bug in priority queue remove function

Pauli noreply at github.com
Wed Nov 8 11:11:14 UTC 2023


  Branch: refs/heads/openssl-3.2
  Home:   https://github.com/openssl/openssl
  Commit: f5f4bc30f303a229d29dea148bc2d289f6fe04e3
      https://github.com/openssl/openssl/commit/f5f4bc30f303a229d29dea148bc2d289f6fe04e3
  Author: Pauli <pauli at openssl.org>
  Date:   2023-11-08 (Wed, 08 Nov 2023)

  Changed paths:
    M ssl/priority_queue.c

  Log Message:
  -----------
  Fix bug in priority queue remove function

The short circuit in the remove function when the element is the last in the
heap, failed to add the removed slot back to the freelist.

Fixes #22644

Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Matt Caswell <matt at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22646)

(cherry picked from commit a03108778044cc0d428ce38084ef6f318446fbe3)


  Commit: 2f563dc3efa7ece9197aaf44cf099bf516ac7421
      https://github.com/openssl/openssl/commit/2f563dc3efa7ece9197aaf44cf099bf516ac7421
  Author: Pauli <pauli at openssl.org>
  Date:   2023-11-08 (Wed, 08 Nov 2023)

  Changed paths:
    M test/priority_queue_test.c

  Log Message:
  -----------
  Add unit test for #22644

Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Matt Caswell <matt at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22646)

(cherry picked from commit 0efcf1384fd320a6235e90d7b078ad89ea504d16)


Compare: https://github.com/openssl/openssl/compare/2e40770aa55c...2f563dc3efa7


More information about the openssl-commits mailing list