commit ee910bfe2486b9d81885b0852e98b3f1ee5be86d
author David Benjamin <davidben@google.com> Tue Jul 25 22:36:00 2017 -0400
committer David Benjamin <davidben@google.com> Fri Jul 28 21:43:13 2017 +0000
tree eb9d8515fe1387089841389ac44fbd229293afea
parent abbf365b6d53ed3445ba2bb40ffb8d5d07245cd4

Use new STACK_OF helpers.

Bug: 132
Change-Id: Ib9bc3ce5f60d0c5bf7922b3d3ccfcd15ef4972a1
Reviewed-on: https://boringssl-review.googlesource.com/18466
Reviewed-by: David Benjamin <davidben@google.com>

ssl/handshake_client.cc

diff --git a/ssl/handshake_client.cc b/ssl/handshake_client.cc
index 32714d1..742e106 100644
--- a/ssl/handshake_client.cc
+++ b/ssl/handshake_client.cc
@@ -599,10 +599,8 @@
   }
 
   if (hs->min_version < TLS1_3_VERSION) {
-    STACK_OF(SSL_CIPHER) *ciphers = SSL_get_ciphers(ssl);
     int any_enabled = 0;
-    for (size_t i = 0; i < sk_SSL_CIPHER_num(ciphers); i++) {
-      const SSL_CIPHER *cipher = sk_SSL_CIPHER_value(ciphers, i);
+    for (const SSL_CIPHER *cipher : SSL_get_ciphers(ssl)) {
       /* Skip disabled ciphers */
       if ((cipher->algorithm_mkey & mask_k) ||
           (cipher->algorithm_auth & mask_a)) {