| commit | 09d68c98c0470521b4bc33e486db7a9f9546c6a5 | [log] [tgz] |
|---|---|---|
| author | Adam Langley <agl@google.com> | Wed Oct 28 18:08:10 2015 -0700 |
| committer | Adam Langley <agl@google.com> | Tue Nov 03 01:15:55 2015 +0000 |
| tree | 9afd459fe55d00051e50bf4260885601e8c4a76c | |
| parent | 2e0901b75f44c81665f2c97365f8c59e51942dc7 [diff] |
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; }