diff options
| author | Matt Caswell <matt@openssl.org> | 2021-07-13 17:19:12 +0100 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2021-07-16 12:20:20 +0200 |
| commit | 21ba77cad67f6a40b051ac9d57069fa58d0658f7 (patch) | |
| tree | 09457a135da93d9dad5f9f047b4e7f4b556e62d2 | |
| parent | e0ad156d22587514b60920143917cdb149734212 (diff) | |
Don't reset the packet pointer in ssl3_setup_read_buffer
Sometimes this function gets called when the buffers have already been
set up. If there is already a partial packet in the read buffer then the
packet pointer will be set to an incorrect value. The packet pointer already
gets reset to the correct value when we first read a packet anyway, so we
don't also need to do it in ssl3_setup_read_buffer.
Fixes #13729
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/16077)
| -rw-r--r-- | ssl/record/ssl3_buffer.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ssl/record/ssl3_buffer.c b/ssl/record/ssl3_buffer.c index 861610a08b..daa175d98c 100644 --- a/ssl/record/ssl3_buffer.c +++ b/ssl/record/ssl3_buffer.c @@ -73,7 +73,6 @@ int ssl3_setup_read_buffer(SSL *s) b->len = len; } - RECORD_LAYER_set_packet(&s->rlayer, &(b->buf[0])); return 1; } |