NIM: resource leak in khui_action_trigger()
authorJeffrey Altman <jaltman@secure-endpoints.com>
Fri, 24 Aug 2007 14:20:31 +0000 (14:20 +0000)
committerJeffrey Altman <jaltman@secure-endpoints.com>
Fri, 24 Aug 2007 14:20:31 +0000 (14:20 +0000)
The khui_action_trigger() function in the Network Identity Manager API
creates a copy of the current user interface context (which contains
information about the credentials and identities that are currently
selected) before triggering the requested action.  Then it will use
the copy to restore the user interface context to its previous state.
A coding error results in the copy of the context to never be
released.  As a result there is a resource leak.

The attached patch fixes the leak by releasing the context once it
is no longer needed.

ticket: new
component: windows

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19852 dc483132-0cff-0310-8789-dd5450dbe970

src/windows/identity/uilib/action.c

index be3860c81afb969d63fa86badb8c6be391dbe8a4..3476428ac2ec0fa6f150edb671472967f8e85fe7 100644 (file)
@@ -497,6 +497,7 @@ khui_action_trigger(khm_int32 action, khui_action_context * ctx) {
 
     if (ctx) {
        khui_context_set_indirect(&save);
+        khui_context_release(&save);
     }
 }