[openssl/openssl] a8086e: stack: Do not add error if pop/shift/value accesse...

Tomáš Mráz noreply at github.com
Fri Oct 21 16:04:53 UTC 2022


  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: a8086e6bfc37355626393751a94bc5c92df7e9d3
      https://github.com/openssl/openssl/commit/a8086e6bfc37355626393751a94bc5c92df7e9d3
  Author: Tomas Mraz <tomas at openssl.org>
  Date:   2022-10-21 (Fri, 21 Oct 2022)

  Changed paths:
    M crypto/conf/conf_def.c
    M crypto/stack/stack.c
    M ssl/ssl_lib.c
    M ssl/statem/statem_srvr.c
    M test/helpers/ssltestlib.c

  Log Message:
  -----------
  stack: Do not add error if pop/shift/value accesses outside of the stack

This partially reverts commit 30eba7f35983a917f1007bce45040c0af3442e42.
This is legitimate use of the stack functions and no error
should be reported apart from the NULL return value.

Fixes #19389

Reviewed-by: Todd Short <todd.short at me.com>
Reviewed-by: Paul Dale <pauli at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19400)


  Commit: 0778364f8ec1f9702e62e6298aaa212ec40b6111
      https://github.com/openssl/openssl/commit/0778364f8ec1f9702e62e6298aaa212ec40b6111
  Author: Tomas Mraz <tomas at openssl.org>
  Date:   2022-10-21 (Fri, 21 Oct 2022)

  Changed paths:
    M doc/man3/DEFINE_STACK_OF.pod

  Log Message:
  -----------
  Document the stack functions that are forgiving

I.e., those that can be called with NULL stack parameter or invalid index.

Reviewed-by: Todd Short <todd.short at me.com>
Reviewed-by: Paul Dale <pauli at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19400)


  Commit: 3a09dfb4f9aace93d2c20d6d1b4968cc583884d6
      https://github.com/openssl/openssl/commit/3a09dfb4f9aace93d2c20d6d1b4968cc583884d6
  Author: Tomas Mraz <tomas at openssl.org>
  Date:   2022-10-21 (Fri, 21 Oct 2022)

  Changed paths:
    M crypto/stack/stack.c

  Log Message:
  -----------
  Errors raised from OPENSSL_sk_set should have ERR_LIB_CRYPTO

Reviewed-by: Todd Short <todd.short at me.com>
Reviewed-by: Paul Dale <pauli at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19400)


Compare: https://github.com/openssl/openssl/compare/fba324204f3b...3a09dfb4f9aa


More information about the openssl-commits mailing list