commit 18a3518e5a94d817740ca89b4ddda63286565d03
author David Benjamin <davidben@google.com> Thu Jun 30 13:46:24 2016 -0400
committer Adam Langley <agl@google.com> Thu Jun 30 22:28:37 2016 +0000
tree c508e0a579a333a5ed48982281279d36036c4114
parent d1e28ad53bec8e1ad6f642792ab663e7e509f67b

Don't allocate a group/curve ID for CECPQ1.

We ended up switching this from a curve to a cipher suite, so the group
ID isn't used. This is in preparation for adding an API for the curve
ID, at which point leaving the protocol constants undefined seems
somewhat bad manners.

Change-Id: Icb8bf4594879dbbc24177551868ecfe89bc2f8c3
Reviewed-on: https://boringssl-review.googlesource.com/8563
Reviewed-by: Adam Langley <agl@google.com>

ssl/handshake_server.c

diff --git a/ssl/handshake_server.c b/ssl/handshake_server.c
index 133278e..e0e1cd7 100644
--- a/ssl/handshake_server.c
+++ b/ssl/handshake_server.c
@@ -1223,8 +1223,8 @@
         goto err;
       }
     } else if (alg_k & SSL_kCECPQ1) {
-      if (!SSL_ECDH_CTX_init(&ssl->s3->tmp.ecdh_ctx, SSL_GROUP_CECPQ1) ||
-          !CBB_add_u16_length_prefixed(&cbb, &child) ||
+      SSL_ECDH_CTX_init_for_cecpq1(&ssl->s3->tmp.ecdh_ctx);
+      if (!CBB_add_u16_length_prefixed(&cbb, &child) ||
           !SSL_ECDH_CTX_offer(&ssl->s3->tmp.ecdh_ctx, &child)) {
         goto err;
       }