diff options
author | Joseph Sutton <josephsutton@catalyst.net.nz> | 2023-05-09 14:53:33 +1200 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2023-05-18 01:03:37 +0000 |
commit | 02e6970ad65c1b1681d9581917130651d3431ede (patch) | |
tree | bcd152fcc023a1ae3b20bc578dd195ff624d0ca2 | |
parent | 2a9d057e8288d00dba9a2f0fe931f35dc370c26a (diff) | |
download | samba-02e6970ad65c1b1681d9581917130651d3431ede.tar.gz |
s4:kdc: Fix leaks
Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
-rw-r--r-- | source4/kdc/db-glue.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/source4/kdc/db-glue.c b/source4/kdc/db-glue.c index bff6d0b165b..be8bd7e3293 100644 --- a/source4/kdc/db-glue.c +++ b/source4/kdc/db-glue.c @@ -2997,6 +2997,7 @@ krb5_error_code samba_kdc_firstkey(krb5_context context, if (!mem_ctx) { ret = ENOMEM; krb5_set_error_message(context, ret, "samba_kdc_firstkey: talloc_named() failed!"); + TALLOC_FREE(priv); return ret; } @@ -3170,6 +3171,7 @@ samba_kdc_check_s4u2proxy(krb5_context context, krb5_set_error_message(context, ret, "samba_kdc_check_s4u2proxy:" " ldb_dn_get_linearized() failed!"); + talloc_free(mem_ctx); return ret; } @@ -3187,6 +3189,7 @@ samba_kdc_check_s4u2proxy(krb5_context context, * delegation target, allow to forward. */ if (target_principal == NULL) { + talloc_free(mem_ctx); return 0; } @@ -3218,6 +3221,7 @@ samba_kdc_check_s4u2proxy(krb5_context context, krb5_set_error_message(context, ret, "samba_kdc_check_s4u2proxy:" " talloc_strdup() failed!"); + talloc_free(mem_ctx); return ret; } @@ -3471,6 +3475,7 @@ NTSTATUS samba_kdc_setup_db_ctx(TALLOC_CTX *mem_ctx, struct samba_kdc_base_conte session_info = system_session(kdc_db_ctx->lp_ctx); if (session_info == NULL) { + talloc_free(kdc_db_ctx); return NT_STATUS_INTERNAL_ERROR; } |