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>
diff --git a/ssl/s3_clnt.c b/ssl/s3_clnt.c index 12eb5e0..5ccce75 100644 --- a/ssl/s3_clnt.c +++ b/ssl/s3_clnt.c
@@ -1456,7 +1456,9 @@ if (CBS_len(&ticket) == 0) { /* 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; return 1; }