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

Tom Yu tlyu at mit.edu
Fri Sep 2 17:04:27 EDT 2016


https://github.com/krb5/krb5/commit/176642a6bac5c6796d4b23124fdc2ed27a0358dc
commit 176642a6bac5c6796d4b23124fdc2ed27a0358dc
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.14.4

 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 13f454f..df337bc 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