feedc0de/wolfssl
1
0
Fork 0
forked from wolfSSL/wolfssl
Code Pull Requests Activity

Merge pull request #4139 from SparkiDev/etm_check_pad

Browse Source 
TLS EtM: check all padding bytes are the same value
This commit is contained in:
David Garske
2021-06-18 08:14:46 -07:00
committed by GitHub
parent 15065175d8 98ce4e901a
commit 5a685ca37e
1 changed files with 27 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
src/internal.c
View File

@ -16110,9 +16110,33 @@ int ProcessReply(WOLFSSL* ssl)
in->buffer + in->idx, in->buffer + in->idx,
ssl->curSize - (word16)digestSz); ssl->curSize - (word16)digestSz);
if (ret == 0) { if (ret == 0) {
ssl->keys.padSz = byte invalid = 0;
in->buffer[in->idx + ssl->curSize - byte padding = (byte)-1;
digestSz - 1]; word32 i;
word32 off = in->idx + ssl->curSize - digestSz - 1;
/* Last of padding bytes - indicates length. */
ssl->keys.padSz = in->buffer[off];
/* Constant time checking of padding - don't leak
* the length of the data.
*/
/* Compare max pad bytes or at most data + pad. */
for (i = 1; i < MAX_PAD_SIZE && off >= i; i++) {
/* Mask on indicates this is expected to be a
* padding byte.
*/
padding &= ctMaskLTE(i, ssl->keys.padSz);
/* When this is a padding byte and not equal
* to length then mask is set.
*/
invalid |= padding &
ctMaskNotEq(in->buffer[off - i],
ssl->keys.padSz);
}
/* If mask is set then there was an error. */
if (invalid) {
ret = DECRYPT_ERROR;
}
ssl->keys.padSz += 1; ssl->keys.padSz += 1;
ssl->keys.decryptedCur = 1; ssl->keys.decryptedCur = 1;
} }