Skip to content

bpo-29870: Free CRL DP with CRL_DIST_POINTS_free()#1085

Closed
tiran wants to merge 1 commit into
python:masterfrom
tiran:bpo-29870-free-crldp
Closed

bpo-29870: Free CRL DP with CRL_DIST_POINTS_free()#1085
tiran wants to merge 1 commit into
python:masterfrom
tiran:bpo-29870-free-crldp

Conversation

@tiran

@tiran tiran commented Apr 11, 2017

Copy link
Copy Markdown
Member

CRL_DIST_POINTS_free() compiles and tests with OpenSSL and libressl
versions

<BuildOpenSSL for 0.9.8zc>
<BuildOpenSSL for 0.9.8zh>
<BuildOpenSSL for 1.0.1u>
<BuildOpenSSL for 1.0.2>
<BuildOpenSSL for 1.0.2k>
<BuildOpenSSL for 1.1.0e>
<BuildLibreSSL for 2.3.10>
<BuildLibreSSL for 2.4.5>
<BuildLibreSSL for 2.5.3>

I have verified that it does no longer leak on OpenSSL 1.0.2k.

CRL_DIST_POINTS_free() compiles and tests with OpenSSL and libressl
versions

<BuildOpenSSL for 0.9.8zc>
<BuildOpenSSL for 0.9.8zh>
<BuildOpenSSL for 1.0.1u>
<BuildOpenSSL for 1.0.2>
<BuildOpenSSL for 1.0.2k>
<BuildOpenSSL for 1.1.0e>
<BuildLibreSSL for 2.3.10>
<BuildLibreSSL for 2.4.5>
<BuildLibreSSL for 2.5.3>

I have verified that it does no longer leak on OpenSSL 1.0.2k.

@pitrou pitrou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@alex

alex commented Apr 11, 2017

Copy link
Copy Markdown
Member

This is a dupe of #526

@Mariatta

Copy link
Copy Markdown
Member

I merged #526. Is this still needed?

@alex

alex commented Apr 15, 2017

Copy link
Copy Markdown
Member

Nope!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants