X-Git-Url: http://git.ozlabs.org/?p=ccan;a=blobdiff_plain;f=ccan%2Ftalloc%2Ftalloc.c;h=ed958a8e5f25fdbeebf83f30b1ea8fef82d018bc;hp=7a2dfab11bd1b7eb493f0ad437bef72b42ac4a97;hb=d7c0cfeb86f8a6474d2fa182c0325e0d3aee0ecd;hpb=d419d5c947a4191aa5dffdcb7c5142ce58457d3d diff --git a/ccan/talloc/talloc.c b/ccan/talloc/talloc.c index 7a2dfab1..ed958a8e 100644 --- a/ccan/talloc/talloc.c +++ b/ccan/talloc/talloc.c @@ -43,7 +43,7 @@ #define ALWAYS_REALLOC 0 -#define MAX_TALLOC_SIZE 0x10000000 +#define MAX_TALLOC_SIZE 0x7FFFFFFF #define TALLOC_MAGIC 0xe814ec70 #define TALLOC_FLAG_FREE 0x01 #define TALLOC_FLAG_LOOP 0x02 @@ -469,7 +469,7 @@ static void *__talloc_steal(const void *new_ctx, const void *ptr) /* internal talloc_free call */ -static inline int _talloc_free(void *ptr) +static inline int _talloc_free(const void *ptr) { struct talloc_chunk *tc; void *oldparent = NULL; @@ -508,7 +508,7 @@ static inline int _talloc_free(void *ptr) return -1; } tc->destructor = (talloc_destructor_t)-1; - if (d(ptr) == -1) { + if (d(discard_const_p(void, ptr)) == -1) { tc->destructor = d; return -1; } @@ -789,6 +789,35 @@ void *_talloc(const void *context, size_t size) return __talloc(context, size); } +static int talloc_destroy_pointer(void ***pptr) +{ + if ((uintptr_t)**pptr < getpagesize()) + TALLOC_ABORT("Double free or invalid talloc_set?"); + /* Invalidate pointer so it can't be used again. */ + **pptr = (void *)1; + return 0; +} + +void _talloc_set(void *ptr, const void *ctx, size_t size, const char *name) +{ + void ***child; + void **pptr = ptr; + + *pptr = talloc_named_const(ctx, size, name); + if (unlikely(!*pptr)) + return; + + child = talloc(*pptr, void **); + if (unlikely(!child)) { + talloc_free(*pptr); + *pptr = NULL; + return; + } + *child = pptr; + talloc_set_name_const(child, "talloc_set destructor"); + talloc_set_destructor(child, talloc_destroy_pointer); +} + /* externally callable talloc_set_name_const() */