URL: https://github.com/SSSD/sssd/pull/714
Author: sumit-bose
 Title: #714: p11_child(openssl): do not free static memory
Action: opened

PR body:
"""
Coverity found a code path where we try to free a static array, this
patch removes it.
"""

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 70912bdce4bc979757aeca164b40e8d75017c269 Mon Sep 17 00:00:00 2001
From: Sumit Bose <sb...@redhat.com>
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.
---
 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 -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
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/sssd-devel@lists.fedorahosted.org

Reply via email to