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/s3_lib.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/s3_lib.c')
| -rw-r--r-- | ssl/s3_lib.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ssl/s3_lib.c b/ssl/s3_lib.c index 1f9f6afff8..3749b2c8f7 100644 --- a/ssl/s3_lib.c +++ b/ssl/s3_lib.c @@ -2799,7 +2799,7 @@ int ssl3_set_handshake_header2(SSL *s, WPACKET *pkt, int htype) { /* Set the content type and 3 bytes for the message len */ if (!WPACKET_put_bytes(pkt, htype, 1) - || !WPACKET_start_sub_packet_len(pkt, 3)) + || !WPACKET_start_sub_packet_u24(pkt)) return 0; return 1; |