Skip to content

Commit 90ca9c4

Browse files
authored
Merge pull request #9864 from JacobBarthelmeh/f11
harden compare of mac with TLS 1.3 finished
2 parents 396b553 + 5117acb commit 90ca9c4

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/tls13.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11214,7 +11214,7 @@ int DoTls13Finished(WOLFSSL* ssl, const byte* input, word32* inOutIdx,
1121411214
if (sniff == NO_SNIFF) {
1121511215
/* Actually check verify data. */
1121611216
if (size > WC_MAX_DIGEST_SIZE ||
11217-
XMEMCMP(input + *inOutIdx, mac, size) != 0){
11217+
ConstantCompare(input + *inOutIdx, mac, size) != 0){
1121811218
WOLFSSL_MSG("Verify finished error on hashes");
1121911219
SendAlert(ssl, alert_fatal, decrypt_error);
1122011220
WOLFSSL_ERROR_VERBOSE(VERIFY_FINISHED_ERROR);

0 commit comments

Comments
 (0)