From df1d42fc1b5fd27a8969819212f51a74b310d909 Mon Sep 17 00:00:00 2001 From: Matthias Dieter Wallnöfer Date: Sun, 29 Nov 2009 14:42:31 +0100 Subject: Revert "s4:registry/util - Don't include the trailing '\0' in the internal data format but add it on the back-conversion to a string" This reverts commit 7d400715e9af2056690c03a1a2f45c7f343fa313. "convert_string_talloc_convenience" does always add the NULL termination. Didn't know that. Thanks Jelmer for pointing out! --- source4/lib/registry/util.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) (limited to 'source4/lib/registry') diff --git a/source4/lib/registry/util.c b/source4/lib/registry/util.c index 764660121f..9409a8f4bc 100644 --- a/source4/lib/registry/util.c +++ b/source4/lib/registry/util.c @@ -56,7 +56,6 @@ _PUBLIC_ char *reg_val_data_string(TALLOC_CTX *mem_ctx, const DATA_BLOB data) { char *ret = NULL; - size_t ret_cnt; if (data.length == 0) return talloc_strdup(mem_ctx, ""); @@ -70,9 +69,7 @@ _PUBLIC_ char *reg_val_data_string(TALLOC_CTX *mem_ctx, data.data, data.length, (void **)&ret, - &ret_cnt, false); - ret = talloc_realloc(mem_ctx, ret, char, ret_cnt + 1); - ret[ret_cnt] = '\0'; + NULL, false); break; case REG_BINARY: ret = data_blob_hex_string_upper(mem_ctx, &data); @@ -136,10 +133,9 @@ _PUBLIC_ bool reg_string_to_val(TALLOC_CTX *mem_ctx, iconv_convenience, CH_UNIX, CH_UTF16, data_str, - strlen(data_str), + strlen(data_str)+1, (void **)&data->data, &data->length, false); - break; case REG_DWORD: { -- cgit