krb5 commit [krb5-1.13]: Fix leak in capaths processing

Tom Yu tlyu at mit.edu
Fri Sep 9 14:48:15 EDT 2016


https://github.com/krb5/krb5/commit/d065fd7505a90227bb9694de5c78d36d53191dfa
commit d065fd7505a90227bb9694de5c78d36d53191dfa
Author: Greg Hudson <ghudson at mit.edu>
Date:   Wed Jun 29 00:01:04 2016 -0400

    Fix leak in capaths processing
    
    In k5_client_realm_path(), free the list of strings returned by
    rtree_capaths_vals().
    
    (cherry picked from commit 08756ce56bddab79afa4ee24b80243d8f822559d)
    
    ticket: 8447
    version_fixed: 1.13.7

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

diff --git a/src/lib/krb5/krb/walk_rtree.c b/src/lib/krb5/krb/walk_rtree.c
index 2b96628..0566a55 100644
--- a/src/lib/krb5/krb/walk_rtree.c
+++ b/src/lib/krb5/krb/walk_rtree.c
@@ -125,7 +125,7 @@ k5_client_realm_path(krb5_context context, const krb5_data *client,
                      const krb5_data *server, krb5_data **rpath_out)
 {
     krb5_error_code retval;
-    char **capvals;
+    char **capvals = NULL;
     size_t i;
     krb5_data *rpath = NULL, d;
 
@@ -160,6 +160,7 @@ k5_client_realm_path(krb5_context context, const krb5_data *client,
     rpath = NULL;
 
 cleanup:
+    profile_free_list(capvals);
     krb5int_free_data_list(context, rpath);
     return retval;
 }


More information about the cvs-krb5 mailing list