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

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

krb5 commit [krb5-1.15]: Fix leaks in gss_inquire_cred_by_oid()

daemon@ATHENA.MIT.EDU (Greg Hudson)
Mon Jul 17 22:59:32 2017

Date: Mon, 17 Jul 2017 22:59:25 -0400
From: Greg Hudson <ghudson@mit.edu>
Message-Id: <201707180259.v6I2xPaK032237@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/c67c28858a557575e2a7a2811b235eb6d639a330
commit c67c28858a557575e2a7a2811b235eb6d639a330
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.
    
    (cherry picked from commit 0d39d46852587d36fcc5024d5766586faba9044a)
    
    ticket: 8559
    version_fixed: 1.15.2

 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