[trafficserver] branch master updated: Fix a build issue with BoringSSL (#6988)

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

maskit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new da88876  Fix a build issue with BoringSSL (#6988)
da88876 is described below

commit da888769f4a83c731e08fdddf5d038cf64fa8b06
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Fri Jul 10 09:04:02 2020 +0900

    Fix a build issue with BoringSSL (#6988)
    
    sk_something_num in OpenSSL returns int but the one in BoringSSL returns size_t.
---
 tests/tools/plugins/ssl_client_verify_test.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/tools/plugins/ssl_client_verify_test.cc b/tests/tools/plugins/ssl_client_verify_test.cc
index 17668b2..c9de34b 100644
--- a/tests/tools/plugins/ssl_client_verify_test.cc
+++ b/tests/tools/plugins/ssl_client_verify_test.cc
@@ -113,7 +113,8 @@ CB_client_verify(TSCont cont, TSEvent event, void *edata)
     STACK_OF(X509) *chain = X509_STORE_CTX_get1_chain(ctx);
     // X509 *cert = X509_STORE_CTX_get_current_cert(ctx);
     bool retval = false;
-    for (int i = 0; i < sk_X509_num(chain) && !retval; i++) {
+    // BoringSSL has sk_X509_num() return size_t.
+    for (int i = 0; i < static_cast<int>(sk_X509_num(chain)) && !retval; i++) {
       auto cert = sk_X509_value(chain, i);
       retval    = check_names(cert);
     }