Correct pointer to be freed #2312

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@levitte
Member
levitte commented Jan 28, 2017 edited

The pointer that was freed in the SSLv2 section of ssl_bytes_to_cipher_list
may have stepped up from its allocated position. Use a pointer that is
guaranteed to point at the start of the allocated block instead.

Checklist
  • CLA is signed
@levitte levitte Correct pointer to be freed
The pointer that was freed in the SSLv2 section of ssl_bytes_to_cipher_list
may have stepped up from its allocated position.  Use a pointer that is
guaranteed to point at the start of the allocated block instead.
1d2705e
@levitte
Member
levitte commented Jan 28, 2017

Fortunately, the error section will never currently be reached, because current preconditions guarantee that the checks will never fail.

@levitte levitte added a commit that referenced this pull request Jan 28, 2017
@levitte levitte Correct pointer to be freed
The pointer that was freed in the SSLv2 section of ssl_bytes_to_cipher_list
may have stepped up from its allocated position.  Use a pointer that is
guaranteed to point at the start of the allocated block instead.

Reviewed-by: Kurt Roeckx <kurt@openssl.org>
(Merged from #2312)
63414e6
@levitte levitte added a commit that referenced this pull request Jan 28, 2017
@levitte levitte Correct pointer to be freed
The pointer that was freed in the SSLv2 section of ssl_bytes_to_cipher_list
may have stepped up from its allocated position.  Use a pointer that is
guaranteed to point at the start of the allocated block instead.

Reviewed-by: Kurt Roeckx <kurt@openssl.org>
(Merged from #2312)
(cherry picked from commit 63414e6)
6f2de02
@levitte
Member
levitte commented Jan 28, 2017

Merged.

master: 63414e6
1.1.0: 6f2de02

@levitte levitte closed this Jan 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment