summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Dieter Wallnöfer <mwallnoefer@yahoo.de>2010-04-17 20:08:15 +0200
committerMatthias Dieter Wallnöfer <mwallnoefer@yahoo.de>2010-04-17 20:10:27 +0200
commitc678563efec82e84c9c86309cbf9495950a02b8c (patch)
tree6ff09a8136b9026beabe783da747ec23725e8001
parent827ccdf6b9c36b7fe0c9b818debd422edceead37 (diff)
downloadsamba-c678563efec82e84c9c86309cbf9495950a02b8c.tar.gz
samba-c678563efec82e84c9c86309cbf9495950a02b8c.tar.bz2
samba-c678563efec82e84c9c86309cbf9495950a02b8c.zip
s4:kdc/db-glue.c - use "TALLOC_FREE" insteal of "talloc_free" for the "priv" context
Also after a free "priv" could be != NULL and may be freed again. This should fix bug #7365.
-rw-r--r--source4/kdc/db-glue.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/source4/kdc/db-glue.c b/source4/kdc/db-glue.c
index 49babeff72..33c4c8cdf6 100644
--- a/source4/kdc/db-glue.c
+++ b/source4/kdc/db-glue.c
@@ -1319,7 +1319,7 @@ static krb5_error_code samba_kdc_seq(krb5_context context,
}
if (ret != 0) {
- talloc_free(priv);
+ TALLOC_FREE(priv);
kdc_db_ctx->seq_ctx = NULL;
} else {
talloc_free(mem_ctx);
@@ -1341,7 +1341,7 @@ krb5_error_code samba_kdc_firstkey(krb5_context context,
int lret;
if (priv) {
- talloc_free(priv);
+ TALLOC_FREE(priv);
kdc_db_ctx->seq_ctx = NULL;
}
@@ -1367,7 +1367,7 @@ krb5_error_code samba_kdc_firstkey(krb5_context context,
ret = krb5_get_default_realm(context, &realm);
if (ret != 0) {
- talloc_free(priv);
+ TALLOC_FREE(priv);
return ret;
}
@@ -1376,7 +1376,7 @@ krb5_error_code samba_kdc_firstkey(krb5_context context,
"(objectClass=user)");
if (lret != LDB_SUCCESS) {
- talloc_free(priv);
+ TALLOC_FREE(priv);
return HDB_ERR_NOENTRY;
}
@@ -1389,7 +1389,7 @@ krb5_error_code samba_kdc_firstkey(krb5_context context,
ret = samba_kdc_seq(context, kdc_db_ctx, entry);
if (ret != 0) {
- talloc_free(priv);
+ TALLOC_FREE(priv);
kdc_db_ctx->seq_ctx = NULL;
} else {
talloc_free(mem_ctx);