krb5 commit: Fix memory leak in test kdcpreauth module

Greg Hudson ghudson at mit.edu
Mon Oct 26 18:19:18 EDT 2015


https://github.com/krb5/krb5/commit/8a3fdba731cb815011bff83ded2ac9f543a1768a
commit 8a3fdba731cb815011bff83ded2ac9f543a1768a
Author: Greg Hudson <ghudson at mit.edu>
Date:   Wed Oct 21 13:23:37 2015 -0400

    Fix memory leak in test kdcpreauth module
    
    In test_verify(), in the second round trip case, free the string
    attribute before returning.

 src/plugins/preauth/test/kdctest.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/plugins/preauth/test/kdctest.c b/src/plugins/preauth/test/kdctest.c
index 82a03b0..026dc68 100644
--- a/src/plugins/preauth/test/kdctest.c
+++ b/src/plugins/preauth/test/kdctest.c
@@ -142,6 +142,7 @@ test_verify(krb5_context context, krb5_data *req_pkt, krb5_kdc_req *request,
         }
         free(str);
         enc_tkt_reply->flags |= TKT_FLG_PRE_AUTH;
+        cb->free_string(context, rock, attr);
         (*respond)(arg, 0, NULL, NULL, NULL);
     } else {
         d = string2data("more");


More information about the cvs-krb5 mailing list