[openssl/openssl] 7eb39e: Make SSL_alloc_buffers() and SSL_free_buffers() wo...

Matt Caswell noreply at github.com
Thu Oct 27 09:54:52 UTC 2022


  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 7eb39ecb299db3eade11946f9385f5dee1d458d3
      https://github.com/openssl/openssl/commit/7eb39ecb299db3eade11946f9385f5dee1d458d3
  Author: Matt Caswell <matt at openssl.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M ssl/record/methods/dtls_meth.c
    M ssl/record/methods/ktls_meth.c
    M ssl/record/methods/recmethod_local.h
    M ssl/record/methods/tls_common.c
    M ssl/record/rec_layer_s3.c
    M ssl/record/record.h
    M ssl/record/recordmethod.h
    M ssl/ssl_lib.c

  Log Message:
  -----------
  Make SSL_alloc_buffers() and SSL_free_buffers() work again

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19472)


  Commit: fba0206da7c0cc68854bb63a6ee9b96a74f4ed7a
      https://github.com/openssl/openssl/commit/fba0206da7c0cc68854bb63a6ee9b96a74f4ed7a
  Author: Matt Caswell <matt at openssl.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M ssl/record/methods/dtls_meth.c
    M ssl/record/methods/ktls_meth.c
    M ssl/record/methods/recmethod_local.h
    M ssl/record/methods/tls_common.c
    M ssl/record/recordmethod.h

  Log Message:
  -----------
  Remove some unused OSSL_RECORD_METHOD functions

Remove two function pointers from the OSSL_RECORD_METHOD. Those functions
were no-ops and were never called.

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19472)


  Commit: ee05588dabeac7b9d034bf16dad122a93d1688a4
      https://github.com/openssl/openssl/commit/ee05588dabeac7b9d034bf16dad122a93d1688a4
  Author: Matt Caswell <matt at openssl.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M test/sslbuffertest.c

  Log Message:
  -----------
  Check whether buffers have actually been allocated/freed

In the sslbuffertest we test the operation of SSL_alloc_buffers() and
SSL_free_buffers(). However this was done entirely using the public API,
and did not confirm that the buffers were actually allocated/freed. We
now extend the test to confirm this.

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19472)


Compare: https://github.com/openssl/openssl/compare/225f94e818d9...ee05588dabea


More information about the openssl-commits mailing list