diff options
author | Jeremy Allison <jra@samba.org> | 2006-12-16 05:02:21 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:16:32 -0500 |
commit | a179d2f495e8dfb9ad30c2b7cec5349cb4e001af (patch) | |
tree | fbf493f2add1b0ca9f567d6bb80f6c8858ab1751 /source3/lib/util_str.c | |
parent | f0c7dc544bccda5eb87f56cbd5a49ca8d8372cb1 (diff) | |
download | samba-a179d2f495e8dfb9ad30c2b7cec5349cb4e001af.tar.gz samba-a179d2f495e8dfb9ad30c2b7cec5349cb4e001af.tar.bz2 samba-a179d2f495e8dfb9ad30c2b7cec5349cb4e001af.zip |
r20208: Change sprintf_append() never to use malloc,
but always use a talloc context.
Thanks to simo for pointing this out.
Jeremy.
(This used to be commit 437cb7c88833d7eab0e3c3dcf175860df74a7a38)
Diffstat (limited to 'source3/lib/util_str.c')
-rw-r--r-- | source3/lib/util_str.c | 18 |
1 files changed, 3 insertions, 15 deletions
diff --git a/source3/lib/util_str.c b/source3/lib/util_str.c index cd52faa52d..ccf0af8b62 100644 --- a/source3/lib/util_str.c +++ b/source3/lib/util_str.c @@ -2458,11 +2458,7 @@ void sprintf_append(TALLOC_CTX *mem_ctx, char **string, ssize_t *len, if (*bufsize == 0) *bufsize = 128; - if (mem_ctx != NULL) - *string = TALLOC_ARRAY(mem_ctx, char, *bufsize); - else - *string = SMB_MALLOC_ARRAY(char, *bufsize); - + *string = TALLOC_ARRAY(mem_ctx, char, *bufsize); if (*string == NULL) goto error; } @@ -2484,13 +2480,8 @@ void sprintf_append(TALLOC_CTX *mem_ctx, char **string, ssize_t *len, } if (increased) { - if (mem_ctx != NULL) { - *string = TALLOC_REALLOC_ARRAY(mem_ctx, *string, char, - *bufsize); - } else { - *string = SMB_REALLOC_ARRAY(*string, char, *bufsize); - } - + *string = TALLOC_REALLOC_ARRAY(mem_ctx, *string, char, + *bufsize); if (*string == NULL) { goto error; } @@ -2503,9 +2494,6 @@ void sprintf_append(TALLOC_CTX *mem_ctx, char **string, ssize_t *len, error: *len = -1; - if (mem_ctx == NULL) { - SAFE_FREE(*string); - } *string = NULL; } |