Expand a comment.

As a follow up to https://boringssl-review.googlesource.com/#/c/6340,
this change expands a comment to note the reason for clearing a flag
that I missed.

Change-Id: Ib3cfecbb330f0ae7c46bf44286f4e6b407159fe8
Reviewed-on: https://boringssl-review.googlesource.com/6393
Reviewed-by: Adam Langley <agl@google.com>
This commit is contained in:
Adam Langley 2015-10-28 18:08:10 -07:00
parent 2e0901b75f
commit 09d68c98c0

View File

@ -1456,7 +1456,9 @@ int ssl3_get_new_session_ticket(SSL *s) {
if (CBS_len(&ticket) == 0) { if (CBS_len(&ticket) == 0) {
/* RFC 5077 allows a server to change its mind and send no ticket after /* RFC 5077 allows a server to change its mind and send no ticket after
* negotiating the extension. Behave as if no ticket was sent. */ * negotiating the extension. The value of |tlsext_ticket_expected| is
* checked in |ssl_update_cache| so is cleared here to avoid an unnecessary
* update. */
s->tlsext_ticket_expected = 0; s->tlsext_ticket_expected = 0;
return 1; return 1;
} }