r24998: Add a function regval_compose() to compose a REGISTRY_VALUE from

input data. Use this function in a first step to refactor
the canonicalization code of smbconf_store_values().

Michael
(This used to be commit f4caa2d7d412e2b9bb2a1ce71514551569ccf373)

1 files changed, 12 insertions, 4 deletions

diff --git a/source3/registry/reg_smbconf.c b/source3/registry/reg_smbconf.c
index e2c7e7ea08..f2a8b6ee8b 100644
--- a/source3/registry/reg_smbconf.c
+++ b/source3/registry/reg_smbconf.c
@@ -167,10 +167,18 @@ static BOOL smbconf_store_values( const char *key, REGVAL_CTR *val )
 
 			DEBUG(10, ("adding canonicalized parameter to "
 				   "container.\n"));
-			res = regval_ctr_addvalue(new_val_ctr, canon_valname,
-						  value->type,
-						  (char *)value_data.data,
-						  value_data.length);
+
+			theval = regval_compose(mem_ctx, canon_valname,
+						value->type,
+						(char *)value_data.data,
+						value_data.length);
+			if (theval == NULL) {
+				DEBUG(10, ("error composing registry value. "
+					   "(no memory?)\n"));
+				TALLOC_FREE(mem_ctx);
+				return False;
+			}
+			res = regval_ctr_copyvalue(new_val_ctr, theval);
 			if (res == 0) {
 				DEBUG(10, ("error calling regval_ctr_addvalue. "
 				      "(no memory?)\n"));