Skip to content
GitLab

Use the default BIO.__del__ rather tha overriding in File

Peter Edwards requested to merge peadar/m2crypto:bioleak into master

File's BIO method was avoiding deleting the raw BIO if the close function already happened, but that only closes the python file, it doesn't free the raw OpenSSL BIO. So just remove del and allow the existing python BIO del method to do its work.

This avoids leaking the raw BIO from methods such as X509.load_cert()

Merge request reports