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

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

krb5 commit [krb5-1.13]: Fix uncommon leak in krb5_init_creds_step()

daemon@ATHENA.MIT.EDU (Tom Yu)
Tue Aug 18 17:15:14 2015

Date: Tue, 18 Aug 2015 17:15:11 -0400
From: Tom Yu <tlyu@mit.edu>
Message-Id: <201508182115.t7ILFBqU019404@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/6ee030430abcfd76294b6599ccaa24c4d36674db
commit 6ee030430abcfd76294b6599ccaa24c4d36674db
Author: Greg Hudson <ghudson@mit.edu>
Date:   Sun Jul 26 13:21:47 2015 -0400

    Fix uncommon leak in krb5_init_creds_step()
    
    Release any previous value of ctx->err_padata before setting it in
    init_creds_step_reply().  It could have a prior value after a realm
    referral or retriable error.
    
    (cherry picked from commit 997eb174f5fd81747ad0ecb671f00c25951931b1)
    
    ticket: 8223
    version_fixed: 1.13.3
    status: resolved

 src/lib/krb5/krb/get_in_tkt.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/lib/krb5/krb/get_in_tkt.c b/src/lib/krb5/krb/get_in_tkt.c
index 2005002..7a50161 100644
--- a/src/lib/krb5/krb/get_in_tkt.c
+++ b/src/lib/krb5/krb/get_in_tkt.c
@@ -1426,6 +1426,8 @@ init_creds_step_reply(krb5_context context,
         ctx->request->client->type == KRB5_NT_ENTERPRISE_PRINCIPAL;
 
     if (ctx->err_reply != NULL) {
+        krb5_free_pa_data(context, ctx->err_padata);
+        ctx->err_padata = NULL;
         code = krb5int_fast_process_error(context, ctx->fast_state,
                                           &ctx->err_reply, &ctx->err_padata,
                                           &retry);
_______________________________________________
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