[29881] in CVS-changelog-for-Kerberos-V5

home help back first fref pref prev next nref lref last post

krb5 commit: Fix leaks in gss_inquire_cred_by_oid()

daemon@ATHENA.MIT.EDU (Greg Hudson)
Mon Mar 13 13:01:40 2017

Date: Mon, 13 Mar 2017 13:01:36 -0400
From: Greg Hudson <ghudson@mit.edu>
Message-Id: <201703131701.v2DH1aGn001401@drugstore.mit.edu>
To: cvs-krb5@mit.edu
Reply-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu

https://github.com/krb5/krb5/commit/0d39d46852587d36fcc5024d5766586faba9044a
commit 0d39d46852587d36fcc5024d5766586faba9044a
Author: Greg Hudson <ghudson@mit.edu>
Date:   Sun Mar 12 12:30:59 2017 -0400

    Fix leaks in gss_inquire_cred_by_oid()
    
    In the mechglue gss_inquire_cred_by_oid(), remove an unnecessary
    allocation of ret_set which is overwritten by the first mechanism's
    result.
    
    ticket: 8559 (new)
    target_version: 1.15-next
    target_version: 1.14-next
    tags: pullup

 src/lib/gssapi/mechglue/g_inq_cred_oid.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/src/lib/gssapi/mechglue/g_inq_cred_oid.c b/src/lib/gssapi/mechglue/g_inq_cred_oid.c
index 4c23dfc..df51b44 100644
--- a/src/lib/gssapi/mechglue/g_inq_cred_oid.c
+++ b/src/lib/gssapi/mechglue/g_inq_cred_oid.c
@@ -85,11 +85,6 @@ gss_inquire_cred_by_oid(OM_uint32 *minor_status,
 
     union_cred = (gss_union_cred_t) cred_handle;
 
-    status = gss_create_empty_buffer_set(minor_status, &ret_set);
-    if (status != GSS_S_COMPLETE) {
-	return status;
-    }
-
     status = GSS_S_UNAVAILABLE;
 
     for (i = 0; i < union_cred->count; i++) {
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5

home help back first fref pref prev next nref lref last post