heimdal: Fix 241482 Resource leak
authorVolker Lendecke <vl@samba.org>
Sun, 10 Nov 2013 08:45:38 +0000 (09:45 +0100)
committerIra Cooper <ira@samba.org>
Mon, 11 Nov 2013 20:04:09 +0000 (21:04 +0100)
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ira Cooper <ira@samba.org>
source4/heimdal/lib/gssapi/krb5/init_sec_context.c

index 5f8b01b7270712e4bdc311f7ef7cc87e3e3a2dc1..0a89ae1f36cfbb04f78763c8b0a064d82c788939 100644 (file)
@@ -137,6 +137,7 @@ _gsskrb5_create_ctx(
     if (kret) {
        *minor_status = kret;
        HEIMDAL_MUTEX_destroy(&ctx->ctx_id_mutex);
+       free(ctx);
        return GSS_S_FAILURE;
     }
 
@@ -145,6 +146,7 @@ _gsskrb5_create_ctx(
        *minor_status = kret;
        krb5_auth_con_free(context, ctx->auth_context);
        HEIMDAL_MUTEX_destroy(&ctx->ctx_id_mutex);
+       free(ctx);
        return GSS_S_FAILURE;
     }
 
@@ -156,7 +158,7 @@ _gsskrb5_create_ctx(
        krb5_auth_con_free(context, ctx->deleg_auth_context);
 
        HEIMDAL_MUTEX_destroy(&ctx->ctx_id_mutex);
-
+       free(ctx);
        return GSS_S_BAD_BINDINGS;
     }
 
@@ -168,7 +170,7 @@ _gsskrb5_create_ctx(
        krb5_auth_con_free(context, ctx->deleg_auth_context);
 
        HEIMDAL_MUTEX_destroy(&ctx->ctx_id_mutex);
-
+       free(ctx);
        return GSS_S_BAD_BINDINGS;
     }