summaryrefslogtreecommitdiff
path: root/source4/lib/talloc
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>2005-01-06 03:20:56 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 13:08:26 -0500
commite159e42d8472f36f51e400e351fc43f2a7dc44f5 (patch)
tree641b7a3f25e77a4b8b1c2724287cd243119c411a /source4/lib/talloc
parentddc10d4d37984246a6547e34a32d629c689c40d1 (diff)
downloadsamba-e159e42d8472f36f51e400e351fc43f2a7dc44f5.tar.gz
samba-e159e42d8472f36f51e400e351fc43f2a7dc44f5.tar.bz2
samba-e159e42d8472f36f51e400e351fc43f2a7dc44f5.zip
r4550: talloc() is now typesafe. It is exactly equivalent to the old talloc_p() macro. Use
talloc_size() if you want the old behaviour. I have kept talloc_p() as an alias for now. Once we change all calls to be plain talloc() then we can remove it. (This used to be commit 2011bbeb841fd6bfccf3d44a49f79203f7f55baa)
Diffstat (limited to 'source4/lib/talloc')
-rw-r--r--source4/lib/talloc/talloc.c4
-rw-r--r--source4/lib/talloc/talloc.h4
-rw-r--r--source4/lib/talloc/talloc_guide.txt40
-rw-r--r--source4/lib/talloc/testsuite.c4
4 files changed, 26 insertions, 26 deletions
diff --git a/source4/lib/talloc/talloc.c b/source4/lib/talloc/talloc.c
index 9e8868191f..bcadf40cfb 100644
--- a/source4/lib/talloc/talloc.c
+++ b/source4/lib/talloc/talloc.c
@@ -857,7 +857,7 @@ char *talloc_strndup(const void *t, const char *p, size_t n)
for (len=0; p[len] && len<n; len++) ;
- ret = talloc(t, len + 1);
+ ret = _talloc(t, len + 1);
if (!ret) { return NULL; }
memcpy(ret, p, len);
ret[len] = 0;
@@ -883,7 +883,7 @@ char *talloc_vasprintf(const void *t, const char *fmt, va_list ap)
len = vsnprintf(NULL, 0, fmt, ap2);
- ret = talloc(t, len+1);
+ ret = _talloc(t, len+1);
if (ret) {
VA_COPY(ap2, ap);
vsnprintf(ret, len+1, fmt, ap2);
diff --git a/source4/lib/talloc/talloc.h b/source4/lib/talloc/talloc.h
index 99a6b7a770..747b1c6ba6 100644
--- a/source4/lib/talloc/talloc.h
+++ b/source4/lib/talloc/talloc.h
@@ -33,11 +33,11 @@ typedef void TALLOC_CTX;
#define __location__ __FILE__ ":" __LINESTR__
/* useful macros for creating type checked pointers */
-#define talloc(ctx, size) talloc_named_const(ctx, size, __location__)
+#define talloc(ctx, type) (type *)talloc_named_const(ctx, sizeof(type), #type)
+#define talloc_p(ctx, type) talloc(ctx, type)
#define talloc_size(ctx, size) talloc_named_const(ctx, size, __location__)
#define talloc_zero(ctx, size) _talloc_zero(ctx, size, __location__)
#define talloc_realloc(ctx, ptr, size) _talloc_realloc(ctx, ptr, size, __location__)
-#define talloc_p(ctx, type) (type *)talloc_named_const(ctx, sizeof(type), #type)
#define talloc_new(ctx) talloc_named_const(ctx, 0, "talloc_new: " __location__)
#define talloc_zero_p(ctx, type) (type *)_talloc_zero(ctx, sizeof(type), #type)
#define talloc_zero_array_p(ctx, type, count) (type *)talloc_zero_array(ctx, sizeof(type), count, __location__)
diff --git a/source4/lib/talloc/talloc_guide.txt b/source4/lib/talloc/talloc_guide.txt
index aa37c60466..af6bdf0275 100644
--- a/source4/lib/talloc/talloc_guide.txt
+++ b/source4/lib/talloc/talloc_guide.txt
@@ -58,11 +58,11 @@ least twice.
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
-void *talloc(const void *context, size_t size);
+(type *)talloc(const void *context, type);
-The talloc() function is the core of the talloc library. It takes a
-memory context, and returns a pointer to a new area of memory of the
-given size.
+The talloc() macro is the core of the talloc library. It takes a
+memory context and a type, and returns a pointer to a new area of
+memory of the given type.
The returned pointer is itself a talloc context, so you can use it as
the context argument to more calls to talloc if you wish.
@@ -74,19 +74,19 @@ well. Alternatively you can free just the child.
The context argument to talloc() can be NULL, in which case a new top
level context is created.
-
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
-void *talloc_p(const void *context, type);
-
-The talloc_p() macro is the equivalent of
+void *talloc_size(const void *context, size_t size);
- (type *)talloc(ctx, sizeof(type))
+The function talloc_size() should be used when you don't have a
+convenient type to pass to talloc(). Unlike talloc(), it is not type
+safe (as it returns a void *), so you are on your own for type checking.
-You should use it in preference to talloc() whenever possible, as it
-provides additional type safety. It also automatically calls the
-talloc_set_name_const() function with the name being a string holding
-the name of the type.
+=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
+void *talloc_p(const void *context, type);
+talloc_p() is a alias for talloc(). It only exists as a backwards
+compatibity macro for code from the bad old days when talloc() was not
+type safe.
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
int talloc_free(void *ptr);
@@ -231,7 +231,7 @@ void *talloc_named(const void *context, size_t size, const char *fmt, ...);
The talloc_named() function creates a named talloc pointer. It is
equivalent to:
- ptr = talloc(context, size);
+ ptr = talloc_size(context, size);
talloc_set_name(ptr, fmt, ....);
@@ -240,7 +240,7 @@ void *talloc_named_const(const void *context, size_t size, const char *name);
This is equivalent to:
- ptr = talloc(context, size);
+ ptr = talloc_size(context, size);
talloc_set_name_const(ptr, name);
@@ -275,7 +275,7 @@ void *talloc_realloc(const void *context, void *ptr, size_t size);
The talloc_realloc() function changes the size of a talloc
pointer. It has the following equivalences:
- talloc_realloc(context, NULL, size) ==> talloc(context, size);
+ talloc_realloc(context, NULL, size) ==> talloc_size(context, size);
talloc_realloc(context, ptr, 0) ==> talloc_free(ptr);
The "context" argument is only used if "ptr" is not NULL, otherwise it
@@ -407,7 +407,7 @@ void *talloc_zero(const void *ctx, size_t size);
The talloc_zero() function is equivalent to:
- ptr = talloc(ctx, size);
+ ptr = talloc_size(ctx, size);
if (ptr) memset(ptr, 0, size);
@@ -416,7 +416,7 @@ void *talloc_memdup(const void *ctx, const void *p, size_t size);
The talloc_memdup() function is equivalent to:
- ptr = talloc(ctx, size);
+ ptr = talloc_size(ctx, size);
if (ptr) memcpy(ptr, p, size);
@@ -425,7 +425,7 @@ char *talloc_strdup(const void *ctx, const char *p);
The talloc_strdup() function is equivalent to:
- ptr = talloc(ctx, strlen(p)+1);
+ ptr = talloc_size(ctx, strlen(p)+1);
if (ptr) memcpy(ptr, p, strlen(p)+1);
This functions sets the name of the new pointer to the passed
@@ -473,7 +473,7 @@ void *talloc_array_p(const void *ctx, type, uint_t count);
The talloc_array_p() macro is equivalent to:
- (type *)talloc(ctx, sizeof(type) * count);
+ (type *)talloc_size(ctx, sizeof(type) * count);
except that it provides integer overflow protection for the multiply,
returning NULL if the multiply overflows.
diff --git a/source4/lib/talloc/testsuite.c b/source4/lib/talloc/testsuite.c
index ced3217105..d46964d9b6 100644
--- a/source4/lib/talloc/testsuite.c
+++ b/source4/lib/talloc/testsuite.c
@@ -523,11 +523,11 @@ static BOOL test_realloc(void)
p1 = talloc_realloc(NULL, p1, 20);
CHECK_SIZE(p1, 20);
- talloc(p1, 0);
+ talloc_new(p1);
p2 = talloc_realloc(p1, NULL, 30);
- talloc(p1, 0);
+ talloc_new(p1);
p2 = talloc_realloc(p1, p2, 40);