diff options
author | Adam Langley <agl@google.com> | 2015-10-29 04:08:10 +0300 |
---|---|---|
committer | Adam Langley <agl@google.com> | 2015-11-03 04:15:55 +0300 |
commit | 09d68c98c0470521b4bc33e486db7a9f9546c6a5 (patch) | |
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>
-rw-r--r-- | ssl/s3_clnt.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ssl/s3_clnt.c b/ssl/s3_clnt.c index 12eb5e09..5ccce754 100644 --- a/ssl/s3_clnt.c +++ b/ssl/s3_clnt.c @@ -1456,7 +1456,9 @@ int ssl3_get_new_session_ticket(SSL *s) { 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; } |