URL: https://github.com/SSSD/sssd/pull/714 Author: sumit-bose Title: #714: p11_child(openssl): do not free static memory Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/714/head:pr714 git checkout pr714
From 410bfa9ea10daa8c76d2c12a29b3d9b908cda11e Mon Sep 17 00:00:00 2001 From: Sumit Bose <[email protected]> Date: Wed, 12 Dec 2018 12:10:42 +0100 Subject: [PATCH] p11_child(openssl): do not free static memory Coverity found a code path where we try to free a static array, this patch removes it. Related to https://pagure.io/SSSD/sssd/issue/3887 --- src/p11_child/p11_child_openssl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/p11_child/p11_child_openssl.c b/src/p11_child/p11_child_openssl.c index 0f8ba3d3c..75e4bef8e 100644 --- a/src/p11_child/p11_child_openssl.c +++ b/src/p11_child/p11_child_openssl.c @@ -881,7 +881,6 @@ static void get_ec_curve_type(CK_FUNCTION_LIST *module, rv = module->C_GetAttributeValue(session, key_handle, &attribute, 1); if (rv != CKR_OK) { - free(attribute.pValue); DEBUG(SSSDBG_OP_FAILURE, "C_GetAttributeValue failed [%lu][%s].\n", rv, p11_kit_strerror(rv));
_______________________________________________ sssd-devel mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/[email protected]
