[openssl/openssl] 20baa2: apps/spkac.c: Check result of ASN1_STRING_set()

Tomáš Mráz noreply at github.com
Fri Aug 18 13:09:15 UTC 2023


  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 20baa24f9f3997a96db7cd176a6ef609afe80cea
      https://github.com/openssl/openssl/commit/20baa24f9f3997a96db7cd176a6ef609afe80cea
  Author: Tomas Mraz <tomas at openssl.org>
  Date:   2023-08-18 (Fri, 18 Aug 2023)

  Changed paths:
    M apps/spkac.c

  Log Message:
  -----------
  apps/spkac.c: Check result of ASN1_STRING_set()

Fixes Coverity 1027256

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


  Commit: 8b940b69457f0dd43496c16afaa01f510f0d8a19
      https://github.com/openssl/openssl/commit/8b940b69457f0dd43496c16afaa01f510f0d8a19
  Author: Tomas Mraz <tomas at openssl.org>
  Date:   2023-08-18 (Fri, 18 Aug 2023)

  Changed paths:
    M test/ssl_old_test.c

  Log Message:
  -----------
  ssl_old_test.c: Check inconsistent values from SSL_get0_alpn_selected()

Fixes Coverity 1534838

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


  Commit: 2b2eedfdd614e4c5e2104acf44da0bcdd5b90ade
      https://github.com/openssl/openssl/commit/2b2eedfdd614e4c5e2104acf44da0bcdd5b90ade
  Author: Tomas Mraz <tomas at openssl.org>
  Date:   2023-08-18 (Fri, 18 Aug 2023)

  Changed paths:
    M crypto/cmp/cmp_genm.c

  Log Message:
  -----------
  cmp_genm.c: Remove superfluous store_ctx != 0 check

This really cannot be ever called with NULL store_ctx
and the check confuses Coverity.

Fixes Coverity 1538865

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


  Commit: 6404d064b8012a2c353603a3b3effa6289313d61
      https://github.com/openssl/openssl/commit/6404d064b8012a2c353603a3b3effa6289313d61
  Author: Tomas Mraz <tomas at openssl.org>
  Date:   2023-08-18 (Fri, 18 Aug 2023)

  Changed paths:
    M crypto/x509/x509_lu.c

  Log Message:
  -----------
  ossl_x509_store_ctx_get_by_subject(): Check return value of X509_STORE_lock()

Fixes Coverity 1539148

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


Compare: https://github.com/openssl/openssl/compare/79cdbe893da0...6404d064b801


More information about the openssl-commits mailing list