sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2025-03-01 04:58:35 +03:00
Code

ldb: pack_format_override option

Browse Source 
For TDB databases, toggling GUID indexing mode will also toggle
pack format version 2. This provides a convenient downgrade path for
Samba databases, but the process doesn't work for MDB databases because
GUID indexing cannot be disabled when the MDB backend is used. This patch
addresses that corner case by providing support for a pack_format_override
option which will force the database to use pack format version 2.

Signed-off-by: Aaron Haslett <aaronhaslett@catalyst.net.nz>
Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>
Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
This commit is contained in:
Aaron Haslett 2019-05-23 19:49:39 +12:00 committed by Andrew Bartlett
parent 68d99187cd
commit 6b4abb9952
2 changed files with 43 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
lib/ldb/ldb_key_value/ldb_kv.c
View File

@ -307,6 +307,12 @@ static int ldb_kv_check_special_dn(struct ldb_module *module,
* and repacks if necessary.
*/
static int ldb_kv_maybe_repack(struct ldb_kv_private *ldb_kv) {
/* Override option taken from ldb options */
if (ldb_kv->pack_format_override != 0) {
ldb_kv->target_pack_format_version =
ldb_kv->pack_format_override;
}
if (ldb_kv->pack_format_version !=
ldb_kv->target_pack_format_version) {
int r;
@ -1933,6 +1939,8 @@ int ldb_kv_init_store(struct ldb_kv_private *ldb_kv,
ldb_kv->pid = getpid();
ldb_kv->pack_format_override = 0;
ldb_kv->module = ldb_module_new(ldb, ldb, name, &ldb_kv_ops);
if (!ldb_kv->module) {
ldb_oom(ldb);
@ -1969,6 +1977,40 @@ int ldb_kv_init_store(struct ldb_kv_private *ldb_kv,
}
}
/*
* Usually the presence of GUID indexing determines the pack format
* we use but in certain circumstances such as downgrading an
* MDB-backed database, we want to override the target pack format.
*
* We set/get opaques here because in the Samba partitions module,
* 'options' are not passed correctly so sub-databases can't see
* the options they need.
*/
{
const char *pack_format_override =
ldb_options_find(ldb, options, "pack_format_override");
if (pack_format_override != NULL) {
int ret;
ldb_kv->pack_format_override =
strtoul(pack_format_override, NULL, 0);
ret = ldb_set_opaque(ldb,
"pack_format_override",
(void *)(intptr_t)ldb_kv->pack_format_override);
if (ret != LDB_SUCCESS) {
talloc_free(ldb_kv->module);
return ldb_module_operr(ldb_kv->module);
}
} else {
/*
* NULL -> 0 is fine, otherwise we get back
* the number we needed
*/
ldb_kv->pack_format_override
= (intptr_t)ldb_get_opaque(ldb,
"pack_format_override");
}
}
/*
* Override full DB scans
*

1
lib/ldb/ldb_key_value/ldb_kv.h
View File

@ -65,6 +65,7 @@ struct ldb_kv_private {
unsigned long long sequence_number;
uint32_t pack_format_version;
uint32_t target_pack_format_version;
uint32_t pack_format_override;
/* the low level tdb seqnum - used to avoid loading BASEINFO when
possible */