[openssl/openssl] 031a8c: siv128.siv128.c: refactor encrypt/decrypt to a sin...
"'Abel Thomas' via openssl-commits" <[email protected]>
| 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: 031a8ccf6496f7b2468b9f4227059291d21683f8
https://github.com/openssl/openssl/commit/031a8ccf6496f7b2468b9f4227059291d21683f8
Author: Abel Thomas <[email protected]>
Date: 2026-08-27 (Thu, 27 Aug 2026)
Changed paths:
M crypto/modes/siv128.c
M test/evp_extra_test.c
Log Message:
-----------
siv128.siv128.c: refactor encrypt/decrypt to a single exit point
`ossl_siv128_encrypt()` and `ossl_siv128_decrypt()` previously set
`ctx->final_ret = 0` only on the success path, leaving it unmodified
on early returns. In speed mode, where the context is reused, this
caused `ossl_siv128_finish()` to return a stale success value after
a failed authentication.
Refactor both functions to use a single err: label exit point.
`final_ret_value` is initialised to -1 and set to 0 only on
success, then assigned to `ctx->final_ret` unconditionally on exit.
Added tests to verify the change.
Fixes #31584
Reviewed-by: Bob Beck <[email protected]>
Reviewed-by: Andrew Dinh <[email protected]>
Merge-date: Thu Aug 27 13:41:41 2026
Merged-from: https://github.com/openssl/openssl/pull/31610
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/95dcb1-031a8c%40github.com.