Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Bug 1618739 - Don't assert fuzzer behavior in SSL_ParseSessionTicket …
…r=jcj

Differential Revision: https://phabricator.services.mozilla.com/D66122

--HG--
extra : moz-landing-system : lando
  • Loading branch information
Kevin Jacobs committed Mar 9, 2020
1 parent c57341c commit 9d0be6f
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion lib/ssl/ssl3exthandle.c
Expand Up @@ -1056,8 +1056,11 @@ ssl_ParseSessionTicket(sslSocket *ss, const SECItem *decryptedTicket,
PORT_SetError(SEC_ERROR_LIBRARY_FAILURE);
return SECFailure;
}
#ifndef UNSAFE_FUZZER_MODE
/* A well-behaving server should only write 0 or 1. */
PORT_Assert(temp == PR_TRUE || temp == PR_FALSE);
parsedTicket->extendedMasterSecretUsed = (PRBool)temp;
#endif
parsedTicket->extendedMasterSecretUsed = temp ? PR_TRUE : PR_FALSE;

rv = ssl3_ExtConsumeHandshake(ss, &temp, 4, &buffer, &len);
if (rv != SECSuccess) {
Expand Down

0 comments on commit 9d0be6f

Please sign in to comment.