bpo-36179: Fix ref leaks in _hashopenssl (GH-12158)

Fix two unlikely reference leaks in _hashopenssl. The leaks only occur in
out-of-memory cases. Thanks to Charalampos Stratakis.

Signed-off-by: Christian Heimes <christian@python.org>



https://bugs.python.org/issue36179
This commit is contained in:
Christian Heimes 2019-03-04 16:45:41 +01:00 committed by Miss Islington (bot)
parent 800d5cd750
commit b7bc283ab6
2 changed files with 10 additions and 6 deletions

View File

@ -0,0 +1,2 @@
Fix two unlikely reference leaks in _hashopenssl. The leaks only occur in
out-of-memory cases.

View File

@ -109,17 +109,18 @@ newEVPobject(PyObject *name)
return NULL;
}
retval->ctx = EVP_MD_CTX_new();
if (retval->ctx == NULL) {
PyErr_NoMemory();
return NULL;
}
/* save the name for .name to return */
Py_INCREF(name);
retval->name = name;
retval->lock = NULL;
retval->ctx = EVP_MD_CTX_new();
if (retval->ctx == NULL) {
Py_DECREF(retval);
PyErr_NoMemory();
return NULL;
}
return retval;
}
@ -182,6 +183,7 @@ EVP_copy_impl(EVPobject *self)
return NULL;
if (!locked_EVP_MD_CTX_copy(newobj->ctx, self)) {
Py_DECREF(newobj);
return _setException(PyExc_ValueError);
}
return (PyObject *)newobj;