From: Andrew Bartlett Date: Fri, 9 Jun 2017 02:09:30 +0000 (+1200) Subject: ldb_tdb: Remove the idxptr DB before we re-index X-Git-Tag: ldb-1.1.31~74 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f6f73f70919e3360197713e378d49e6eecfe2f1c;p=thirdparty%2Fsamba.git ldb_tdb: Remove the idxptr DB before we re-index We do not want the cache or any of the values in it, we want to read the real DB @INDEX: records. This matters if a re-index is tiggered in the same transaction as the modify of the values in the index. Otherwise we won't see the old index record (it will not show up in the tdb_traverse) and so fail to remove it. That in turn can cause a spurious unqiue index violation. Signed-off-by: Andrew Bartlett Reviewed-by: Garming Sam --- diff --git a/lib/ldb/ldb_tdb/ldb_index.c b/lib/ldb/ldb_tdb/ldb_index.c index e1e54ba38db..76f3cb392e2 100644 --- a/lib/ldb/ldb_tdb/ldb_index.c +++ b/lib/ldb/ldb_tdb/ldb_index.c @@ -1655,6 +1655,18 @@ int ltdb_reindex(struct ldb_module *module) return LDB_ERR_OPERATIONS_ERROR; } + /* + * Ensure we read (and so remove) the entries from the real + * DB, no values stored so far are any use as we want to do a + * re-index + */ + ltdb_index_transaction_cancel(module); + + ret = ltdb_index_transaction_start(module); + if (ret != LDB_SUCCESS) { + return ret; + } + /* first traverse the database deleting any @INDEX records by * putting NULL entries in the in-memory tdb */