lib/ldb: Use talloc_memdup() because we know the length of the attribute already

BUG: https://bugzilla.samba.org/show_bug.cgi?id=11602
Signed-off-by: Adrian Cochrane <adrianc@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>

diff --git a/lib/ldb/common/ldb_pack.c b/lib/ldb/common/ldb_pack.c
index 49d226ce06667823492ad7d0ef36304e8beddf18..7970b9d3ecb8325b4a46f1da46236810009cbb50 100644 (file)
--- a/lib/ldb/common/ldb_pack.c
+++ b/lib/ldb/common/ldb_pack.c
@@ -373,7 +373,7 @@ int ldb_unpack_data_only_attr_list(struct ldb_context *ldb,
                        }
                }
                element = &message->elements[nelem];
-               element->name = talloc_strndup(message->elements, attr, attr_len);
+               element->name = talloc_memdup(message->elements, attr, attr_len+1);
 
                if (element->name == NULL) {
                        errno = ENOMEM;