Merge pull request #1789 from JacobBarthelmeh/Fuzzer

sanity check on buffer before clearing it
This commit is contained in:
toddouska
2018-08-29 16:10:56 -07:00
committed by GitHub

View File

@ -20019,7 +20019,9 @@ exit_scke:
#endif #endif
/* No further need for PMS */ /* No further need for PMS */
ForceZero(ssl->arrays->preMasterSecret, ssl->arrays->preMasterSz); if (ssl->arrays->preMasterSecret != NULL) {
ForceZero(ssl->arrays->preMasterSecret, ssl->arrays->preMasterSz);
}
ssl->arrays->preMasterSz = 0; ssl->arrays->preMasterSz = 0;
/* Final cleanup */ /* Final cleanup */
@ -25264,7 +25266,9 @@ static int DoSessionTicket(WOLFSSL* ssl, const byte* input, word32* inOutIdx,
#endif /* WOLFSSL_ASYNC_CRYPT */ #endif /* WOLFSSL_ASYNC_CRYPT */
/* Cleanup PMS */ /* Cleanup PMS */
ForceZero(ssl->arrays->preMasterSecret, ssl->arrays->preMasterSz); if (ssl->arrays->preMasterSecret != NULL) {
ForceZero(ssl->arrays->preMasterSecret, ssl->arrays->preMasterSz);
}
ssl->arrays->preMasterSz = 0; ssl->arrays->preMasterSz = 0;
/* Final cleanup */ /* Final cleanup */