Merge pull request #7492 from JacobBarthelmeh/porting

remove assumption of struct layout
pull/7499/head
Sean Parkinson 2024-05-02 07:14:06 +10:00 committed by GitHub
commit 75b178f666
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 7 additions and 1 deletions

View File

@ -11273,7 +11273,13 @@ static int GetRecordHeader(WOLFSSL* ssl, word32* inOutIdx,
ssl->fuzzerCb(ssl, ssl->buffers.inputBuffer.buffer + *inOutIdx,
RECORD_HEADER_SZ, FUZZ_HEAD, ssl->fuzzerCtx);
#endif
XMEMCPY(rh, ssl->buffers.inputBuffer.buffer + *inOutIdx, RECORD_HEADER_SZ);
/* Set explicitly rather than make assumptions on struct layout */
rh->type = ssl->buffers.inputBuffer.buffer[*inOutIdx];
rh->pvMajor = ssl->buffers.inputBuffer.buffer[*inOutIdx + 1];
rh->pvMinor = ssl->buffers.inputBuffer.buffer[*inOutIdx + 2];
rh->length[0] = ssl->buffers.inputBuffer.buffer[*inOutIdx + 3];
rh->length[1] = ssl->buffers.inputBuffer.buffer[*inOutIdx + 4];
*inOutIdx += RECORD_HEADER_SZ;
ato16(rh->length, size);
}