diff options
author | Matt Caswell <matt@openssl.org> | 2016-09-09 00:13:41 +0100 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2016-09-13 09:41:21 +0100 |
commit | de451856f08364ad6c6659b6eacbe820edc2aab9 (patch) | |
tree | 9906cf7b996f9c1316e800390acc96e21e20f040 /ssl/t1_ext.c | |
parent | 6ae4f5e087d204e02a5dc88ea905cca9d144a30d (diff) |
Address WPACKET review comments
A few style tweaks here and there. The main change is that curr and
packet_len are now offsets into the buffer to account for the fact that
the pointers can change if the buffer grows. Also dropped support for the
WPACKET_set_packet_len() function. I thought that was going to be needed
but so far it hasn't been. It doesn't really work any more due to the
offsets change.
Reviewed-by: Rich Salz <rsalz@openssl.org>
Diffstat (limited to 'ssl/t1_ext.c')
-rw-r--r-- | ssl/t1_ext.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ssl/t1_ext.c b/ssl/t1_ext.c index cf4f5b0b25..664906c1c2 100644 --- a/ssl/t1_ext.c +++ b/ssl/t1_ext.c @@ -172,7 +172,7 @@ int custom_ext_add(SSL *s, int server, WPACKET *pkt, int *al) } if (!WPACKET_put_bytes(pkt, meth->ext_type, 2) - || !WPACKET_start_sub_packet_len(pkt, 2) + || !WPACKET_start_sub_packet_u16(pkt) || (outlen > 0 && !WPACKET_memcpy(pkt, out, outlen)) || !WPACKET_close(pkt)) { *al = SSL_AD_INTERNAL_ERROR; |