[openssl/openssl] f80910: Coverity 1503321 & 1503327: dereference after null...

pauli noreply at reply.github.openssl.org
Sun Jul 17 07:51:40 UTC 2022


  Branch: refs/heads/master
  Home:   https://github.openssl.org/openssl/openssl
  Commit: f80910390cb882f346fe59c9803fc914b9c367c2
      https://github.openssl.org/openssl/openssl/commit/f80910390cb882f346fe59c9803fc914b9c367c2
  Author: Pauli <pauli at openssl.org>
  Date:   2022-07-17 (Sun, 17 Jul 2022)

  Changed paths:
    M crypto/provider_core.c

  Log Message:
  -----------
  Coverity 1503321 & 1503327: dereference after null check

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


  Commit: ee8db8c5fb5b091f48d29914126d35a7e29cdcf2
      https://github.openssl.org/openssl/openssl/commit/ee8db8c5fb5b091f48d29914126d35a7e29cdcf2
  Author: Pauli <pauli at openssl.org>
  Date:   2022-07-17 (Sun, 17 Jul 2022)

  Changed paths:
    M crypto/evp/evp_lib.c
    M crypto/evp/evp_rand.c
    M crypto/evp/exchange.c
    M crypto/evp/kdf_lib.c
    M crypto/evp/kem.c
    M crypto/evp/keymgmt_meth.c
    M crypto/evp/mac_lib.c
    M crypto/evp/p_lib.c
    M crypto/evp/signature.c

  Log Message:
  -----------
  evp: make all _is_a functions accept and handle a NULL argument

Makes life easier for callers.

Fixes Coverity 1503326

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


  Commit: 358103b4a651ab3f392f088d86cd30469dccce2e
      https://github.openssl.org/openssl/openssl/commit/358103b4a651ab3f392f088d86cd30469dccce2e
  Author: Pauli <pauli at openssl.org>
  Date:   2022-07-17 (Sun, 17 Jul 2022)

  Changed paths:
    M crypto/bio/bss_dgram.c

  Log Message:
  -----------
  Coverity 1506566: unchecked return value

There isn't much else that can be done here unfortunately.

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


Compare: https://github.openssl.org/openssl/openssl/compare/98b183d3c65e...358103b4a651


More information about the openssl-commits mailing list