From fd5174e88ca1727a91d6dc9bf9bd898ff9087fe8 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Wed, 4 Nov 2009 17:42:53 +1100 Subject: lib/util Split data_blob_hex_string() into upper and lower Rather than have a repeat of the bugs we found at the plugfest where hexidecimal strings must be in upper or lower case in particular places, ensure that each caller chooses which case they want. This reverts most of the callers back to upper case, as things were before tridge's patch. The critical call in the extended DN code is of course handled in lower case. Andrew Bartlett --- source4/lib/ldb-samba/ldif_handlers.c | 2 +- source4/lib/registry/util.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'source4/lib') diff --git a/source4/lib/ldb-samba/ldif_handlers.c b/source4/lib/ldb-samba/ldif_handlers.c index 9467bef873..d8edeb0beb 100644 --- a/source4/lib/ldb-samba/ldif_handlers.c +++ b/source4/lib/ldb-samba/ldif_handlers.c @@ -784,7 +784,7 @@ static int ldif_write_replUpToDateVector(struct ldb_context *ldb, void *mem_ctx, static int extended_dn_write_hex(struct ldb_context *ldb, void *mem_ctx, const struct ldb_val *in, struct ldb_val *out) { - *out = data_blob_string_const(data_blob_hex_string(mem_ctx, in)); + *out = data_blob_string_const(data_blob_hex_string_lower(mem_ctx, in)); if (!out->data) { return -1; } diff --git a/source4/lib/registry/util.c b/source4/lib/registry/util.c index a1897eff2e..038b23fc87 100644 --- a/source4/lib/registry/util.c +++ b/source4/lib/registry/util.c @@ -68,7 +68,7 @@ _PUBLIC_ char *reg_val_data_string(TALLOC_CTX *mem_ctx, (void **)&ret, NULL, false); return ret; case REG_BINARY: - ret = data_blob_hex_string(mem_ctx, &data); + ret = data_blob_hex_string_upper(mem_ctx, &data); return ret; case REG_DWORD: if (*(int *)data.data == 0) -- cgit