[openssl/openssl] df6e80: statem: fix missing fatal if valid_flags mfail in ...

"'Jakub Zelenka' via openssl-commits" <[email protected]>
Newsgroups gmane.comp.encryption.openssl.cvs
Message-ID <openssl/openssl/push/refs/heads/openssl-3.6/[email protected]>
  Branch: refs/heads/openssl-3.6
  Home:   https://github.com/openssl/openssl
  Commit: df6e8057821cea88250dc0cf9d1b30f4ea5a9eed
      https://github.com/openssl/openssl/commit/df6e8057821cea88250dc0cf9d1b30f4ea5a9eed
  Author: Jakub Zelenka <[email protected]>
  Date:   2026-06-03 (Wed, 03 Jun 2026)

  Changed paths:
    M ssl/statem/statem_clnt.c

  Log Message:
  -----------
  statem: fix missing fatal if valid_flags mfail in process cert req

It is a contract of tls process functions to trigger fatal error if they
fail. This is not being done in checking result of s->s3.tmp.valid_flags
allocation. If this happens, it triggers alert in read_state_machine()
for READ_STATE_BODY state that calls this process function. It calls
check_fatal() if MSG_PROCESS_ERROR is returned and the assert in it
fails because no error is triggered.

The fix just adds the fatal and also uses MSG_PROCESS_ERROR macro as
return value instead of hard coded 0.

Reviewed-by: Milan Broz <[email protected]>
Reviewed-by: Paul Yang <[email protected]>
MergeDate: Wed Jun  3 11:34:13 2026
(Merged from https://github.com/openssl/openssl/pull/31338)



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/openssl-3.6/fda996-df6e80%40github.com.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.