[openssl/openssl] eac8ed: Add return check to OSSL_DECODER_up_ref

"'Neil Horman' via openssl-commits" <[email protected]> Thu, 23 Jul 2026 01:25:36 -0700
Newsgroups gmane.comp.encryption.openssl.cvs
Message-ID <openssl/openssl/push/refs/heads/master/[email protected]>
  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: eac8ed4825213a80644bb5ea15234a59daa57352
      https://github.com/openssl/openssl/commit/eac8ed4825213a80644bb5ea15234a59daa57352
  Author: Neil Horman <[email protected]>
  Date:   2026-07-23 (Thu, 23 Jul 2026)

  Changed paths:
    M crypto/encode_decode/decoder_meth.c

  Log Message:
  -----------
  Add return check to OSSL_DECODER_up_ref

Coverity flagged a location where we didn't check this return code, fix
it up.

Fixes https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1696759

Reviewed-by: Milan Broz <[email protected]>
Reviewed-by: Andrew Dinh <[email protected]>
Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
MergeDate: Thu Jul 23 08:24:22 2026
(Merged from https://github.com/openssl/openssl/pull/31992)


  Commit: 9375418a8758b9229182da5d5922342c08bc7c74
      https://github.com/openssl/openssl/commit/9375418a8758b9229182da5d5922342c08bc7c74
  Author: Neil Horman <[email protected]>
  Date:   2026-07-23 (Thu, 23 Jul 2026)

  Changed paths:
    M crypto/asn1/a_strex.c

  Log Message:
  -----------
  use safe_math on do_name_ex

Coverity caught some potential overflows in do_name_ex.  Seems unlikely
to occur, but lets just do all the adds under safe_math to ensure we
don't overflow our buffer length.

Fixes https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1695278

Reviewed-by: Milan Broz <[email protected]>
Reviewed-by: Andrew Dinh <[email protected]>
Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
MergeDate: Thu Jul 23 08:24:23 2026
(Merged from https://github.com/openssl/openssl/pull/31992)


  Commit: 05b241cb094941c7f83ebd94d5516648d82bc2d9
      https://github.com/openssl/openssl/commit/05b241cb094941c7f83ebd94d5516648d82bc2d9
  Author: Neil Horman <[email protected]>
  Date:   2026-07-23 (Thu, 23 Jul 2026)

  Changed paths:
    M test/statem_clnt_construct_test.c

  Log Message:
  -----------
  Fix missing NULL check in statem_clnt_construct_test

Fixes https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1695905

Reviewed-by: Milan Broz <[email protected]>
Reviewed-by: Andrew Dinh <[email protected]>
Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
MergeDate: Thu Jul 23 08:24:25 2026
(Merged from https://github.com/openssl/openssl/pull/31992)


Compare: https://github.com/openssl/openssl/compare/ba2f3b557a66...05b241cb0949

To unsubscribe from these emails, change your notification settings at https://github.com/openssl/openssl/settings/notifications

-- 
You received this message because you are subscribed to the Google Groups "openssl-commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
To view this discussion visit https://groups.google.com/a/openssl.org/d/msgid/openssl-commits/openssl/openssl/push/refs/heads/master/ba2f3b-05b241%40github.com.