ldb: Indicate that the ltdb_dn_list_sort() in list_union is a bit subtle.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>

diff --git a/lib/ldb/ldb_tdb/ldb_index.c b/lib/ldb/ldb_tdb/ldb_index.c
index dbe59837f6e0cd07f08c538e92761b24052b7715..1a5867b7a0dbbf4510347496bbedbc655f723a9b 100644 (file)
--- a/lib/ldb/ldb_tdb/ldb_index.c
+++ b/lib/ldb/ldb_tdb/ldb_index.c
@@ -1310,6 +1310,9 @@ static bool list_union(struct ldb_context *ldb,
        /*
         * Sort the lists (if not in GUID DN mode) so we can do
         * the de-duplication during the merge
+        *
+        * NOTE: This can sort the in-memory index values, as list or
+        * list2 might not be a copy!
         */
        ltdb_dn_list_sort(ltdb, list);
        ltdb_dn_list_sort(ltdb, list2);