kris/boringssl
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Convert ssl3_send_next_proto to CBB.

Browse Source 
BUG=468889

Change-Id: I841b2816ba47c8c1129e333012272902a52cafd3
Reviewed-on: https://boringssl-review.googlesource.com/6200
Reviewed-by: Adam Langley <alangley@gmail.com>
This commit is contained in:
David Benjamin 2015-10-10 14:59:09 -04:00 committed by Adam Langley
parent 5638046cd7
commit e97b96224c
2 changed files with 28 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ssl/internal.h
View File

@ -1115,7 +1115,7 @@ int ssl_do_client_cert_cb(SSL *s, X509 **px509, EVP_PKEY **ppkey);
int ssl3_send_client_key_exchange(SSL *s);
int ssl3_get_server_key_exchange(SSL *s);
int ssl3_get_server_certificate(SSL *s);
int ssl3_send_next_proto(SSL *s);
int ssl3_send_next_proto(SSL *ssl);
int ssl3_send_channel_id(SSL *s);
int ssl3_verify_server_cert(SSL *s);

40
ssl/s3_clnt.c
View File

@ -2067,29 +2067,35 @@ int ssl3_send_client_certificate(SSL *s) {
return ssl_do_write(s);
}
int ssl3_send_next_proto(SSL *s) {
unsigned int len, padding_len;
uint8_t *d, *p;
int ssl3_send_next_proto(SSL *ssl) {
if (ssl->state == SSL3_ST_CW_NEXT_PROTO_B) {
return ssl_do_write(ssl);
}
if (s->state == SSL3_ST_CW_NEXT_PROTO_A) {
len = s->next_proto_negotiated_len;
padding_len = 32 - ((len + 2) % 32);
assert(ssl->state == SSL3_ST_CW_NEXT_PROTO_A);
d = p = ssl_handshake_start(s);
*(p++) = len;
memcpy(p, s->next_proto_negotiated, len);
p += len;
*(p++) = padding_len;
memset(p, 0, padding_len);
p += padding_len;
static const uint8_t kZero[32] = {0};
size_t padding_len = 32 - ((ssl->next_proto_negotiated_len + 2) % 32);
if (!ssl_set_handshake_header(s, SSL3_MT_NEXT_PROTO, p - d)) {
CBB cbb, child;
size_t length;
CBB_zero(&cbb);
if (!CBB_init_fixed(&cbb, ssl_handshake_start(ssl),
ssl->init_buf->max - SSL_HM_HEADER_LENGTH(ssl)) ||
!CBB_add_u8_length_prefixed(&cbb, &child) ||
!CBB_add_bytes(&child, ssl->next_proto_negotiated,
ssl->next_proto_negotiated_len) ||
!CBB_add_u8_length_prefixed(&cbb, &child) ||
!CBB_add_bytes(&child, kZero, padding_len) ||
!CBB_finish(&cbb, NULL, &length) ||
!ssl_set_handshake_header(ssl, SSL3_MT_NEXT_PROTO, length)) {
OPENSSL_PUT_ERROR(SSL, ERR_R_INTERNAL_ERROR);
CBB_cleanup(&cbb);
return -1;
}
s->state = SSL3_ST_CW_NEXT_PROTO_B;
}
return ssl_do_write(s);
ssl->state = SSL3_ST_CW_NEXT_PROTO_B;
return ssl_do_write(ssl);
}
int ssl3_send_channel_id(SSL *s) {