krb5 commit: Fix leak in kadm5_hook interface

Greg Hudson ghudson at mit.edu
Wed Aug 10 13:51:42 EDT 2016


https://github.com/krb5/krb5/commit/40f25ae58415304006dd66ad55eeeb7505d1a816
commit 40f25ae58415304006dd66ad55eeeb7505d1a816
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.
    
    ticket: 8445 (new)
    target_version: 1.14-next
    target_version: 1.13-next
    tags: pullup

 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