Skip to content

Commit 1d49f41

Browse files
authored
Merge pull request #9914 from julek-wolfssl/fenrir/30
Make sure size check doesn't underflow
2 parents 467f16f + 5e22d04 commit 1d49f41

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/internal.c

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22880,8 +22880,10 @@ static int DoProcessReplyEx(WOLFSSL* ssl, int allowSocketErr)
2288022880
#ifdef WOLFSSL_TLS13
2288122881
if (IsAtLeastTLSv1_3(ssl->version)) {
2288222882
tooLong = ssl->curSize > MAX_TLS13_ENC_SZ;
22883-
tooLong |= ssl->curSize - ssl->specs.aead_mac_size >
22883+
if (ssl->specs.aead_mac_size < ssl->curSize) {
22884+
tooLong |= ssl->curSize - ssl->specs.aead_mac_size >
2288422885
MAX_TLS13_PLAIN_SZ;
22886+
}
2288522887
}
2288622888
#endif
2288722889
#ifdef WOLFSSL_EXTRA_ALERTS

0 commit comments

Comments
 (0)