summaryrefslogtreecommitdiff
path: root/source3/lib
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2002-12-14 00:57:01 +0000
committerJeremy Allison <jra@samba.org>2002-12-14 00:57:01 +0000
commita030b19767e801d480705855ff9549b924ad8259 (patch)
tree9152f5f8e9200184a94c5ad3881ba9e3aa5f5c93 /source3/lib
parent5f9348d4fcff534e6efb129fcab5d81ac4f9ba6e (diff)
downloadsamba-a030b19767e801d480705855ff9549b924ad8259.tar.gz
samba-a030b19767e801d480705855ff9549b924ad8259.tar.bz2
samba-a030b19767e801d480705855ff9549b924ad8259.zip
The name pointer in the talloc context must not be a talloced entry as
calling talloc_destroy_pool(as we do sometimes) will destroy it. Jeremy. (This used to be commit 63f344e27be5aaf2204899fea7d53a7302001108)
Diffstat (limited to 'source3/lib')
-rw-r--r--source3/lib/talloc.c15
1 files changed, 13 insertions, 2 deletions
diff --git a/source3/lib/talloc.c b/source3/lib/talloc.c
index 0f293e1725..5fffe58f9f 100644
--- a/source3/lib/talloc.c
+++ b/source3/lib/talloc.c
@@ -138,6 +138,7 @@ TALLOC_CTX *talloc_init(void)
* Create a new talloc context, with a name specifying its purpose.
* Please call this in preference to talloc_init().
**/
+
TALLOC_CTX *talloc_init_named(char const *fmt, ...)
{
TALLOC_CTX *t;
@@ -145,9 +146,18 @@ TALLOC_CTX *talloc_init(void)
t = talloc_init();
if (t && fmt) {
+ /*
+ * t->name must not be talloced.
+ * as destroying the pool would destroy it. JRA.
+ */
+ t->name = NULL;
va_start(ap, fmt);
- t->name = talloc_vasprintf(t, fmt, ap);
+ vasprintf(&t->name, fmt, ap);
va_end(ap);
+ if (!t->name) {
+ talloc_destroy(t);
+ t = NULL;
+ }
}
return t;
@@ -234,6 +244,7 @@ void talloc_destroy(TALLOC_CTX *t)
talloc_destroy_pool(t);
talloc_disenroll(t);
+ SAFE_FREE(t->name);
memset(t, 0, sizeof(TALLOC_CTX));
SAFE_FREE(t);
}
@@ -411,7 +422,7 @@ char *talloc_describe_all(TALLOC_CTX *rt)
if (it->name)
fstrcpy(what, it->name);
else
- slprintf(what, sizeof what, "@%p", it);
+ slprintf(what, sizeof(what), "@%p", it);
s = talloc_asprintf_append(rt, s, "%-40s %8u %8u\n",
what,