commit 50e7ea5f09d15a65f5dd0b63b64504d6d4815001
author Anton Bikineev <bikineev@chromium.org> Sun Jan 23 22:35:48 2022 +0100
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Jan 24 17:37:48 2022 +0000
tree 9116954a43c6d2eb8699485a1540e9ca60874020
parent 960ddfee4ebfb699ca7e245a2e654e231ea8e4a8

LSC: Apply clang-tidy's modernize-use-bool-literals to boringssl

The check finds implicit conversions of integer literals to bools:
  bool b1 = 1;
  bool b2 = static_cast<bool>(1);
and transforms them to:
  bool b1 = true;
  bool b2 = true;

Bug: chromium:1290142
Change-Id: I15579e28f544d07b331a230b70a8278e0651150d
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/51085
Reviewed-by: David Benjamin <davidben@google.com>
Commit-Queue: David Benjamin <davidben@google.com>

ssl/handshake_server.cc

diff --git a/ssl/handshake_server.cc b/ssl/handshake_server.cc
index fdf9511..1d03c55 100644
--- a/ssl/handshake_server.cc
+++ b/ssl/handshake_server.cc
@@ -1311,7 +1311,7 @@
     hs->new_session->verify_result = X509_V_OK;
   } else if (hs->config->retain_only_sha256_of_client_certs) {
     // The hash will have been filled in.
-    hs->new_session->peer_sha256_valid = 1;
+    hs->new_session->peer_sha256_valid = true;
   }
 
   ssl->method->next_message(ssl);