krb5 commit [krb5-1.13]: Fix leak in kadm5_hook interface

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


https://github.com/krb5/krb5/commit/f61212ac11be67de21d724895095628b1062364f
commit f61212ac11be67de21d724895095628b1062364f
Author: Greg Hudson <ghudson at mit.edu>
Date:   Tue Jun 28 22:15:17 2016 -0400

    Fix leak in kadm5_hook interface
    
    In k5_kadm5_hook_free_handles(), free each handle structure as well as
    its module data.
    
    (cherry picked from commit 40f25ae58415304006dd66ad55eeeb7505d1a816)
    
    ticket: 8445
    version_fixed: 1.13.7

 src/lib/kadm5/srv/kadm5_hook.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/lib/kadm5/srv/kadm5_hook.c b/src/lib/kadm5/srv/kadm5_hook.c
index 62f3bff..8e68e8c 100644
--- a/src/lib/kadm5/srv/kadm5_hook.c
+++ b/src/lib/kadm5/srv/kadm5_hook.c
@@ -104,6 +104,7 @@ k5_kadm5_hook_free_handles(krb5_context context, kadm5_hook_handle *handles)
         handle = *hp;
         if (handle->vt.fini != NULL)
             handle->vt.fini(context, handle->data);
+        free(handle);
     }
     free(handles);
 }


More information about the cvs-krb5 mailing list