diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2011-09-14 07:10:13 +0930 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2011-09-14 07:10:13 +0930 |
commit | e69c90e5d59027695f158751ed7922d2389d2b5b (patch) | |
tree | 7a0755d7aa3ccb0f7ebfb05d3c126552e0d55980 /lib/tdb2/test/run-35-convert.c | |
parent | 34c2d1658a89462e9b34210cb19fe9ab33bc2194 (diff) | |
download | samba-e69c90e5d59027695f158751ed7922d2389d2b5b.tar.gz samba-e69c90e5d59027695f158751ed7922d2389d2b5b.tar.bz2 samba-e69c90e5d59027695f158751ed7922d2389d2b5b.zip |
tdb2: enforce TDB_CONVERT
If the caller actually specifies TDB_CONVERT, fail if the TDB does not
need conversion. It's unusual for the user to specify TDB_CONVERT
(it's auto-detected) but better to be strict.
(Imported from CCAN commit 3e3ee8b0c1e77340ab9e8678c2d23646d8cdb627)
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'lib/tdb2/test/run-35-convert.c')
-rw-r--r-- | lib/tdb2/test/run-35-convert.c | 61 |
1 files changed, 61 insertions, 0 deletions
diff --git a/lib/tdb2/test/run-35-convert.c b/lib/tdb2/test/run-35-convert.c new file mode 100644 index 0000000000..0300369194 --- /dev/null +++ b/lib/tdb2/test/run-35-convert.c @@ -0,0 +1,61 @@ +#include <ccan/tdb2/private.h> +#include <ccan/failtest/failtest_override.h> +#include <ccan/tdb2/tdb.c> +#include <ccan/tdb2/open.c> +#include <ccan/tdb2/free.c> +#include <ccan/tdb2/lock.c> +#include <ccan/tdb2/io.c> +#include <ccan/tdb2/hash.c> +#include <ccan/tdb2/transaction.c> +#include <ccan/tdb2/check.c> +#include <ccan/tap/tap.h> +#include <ccan/failtest/failtest.h> +#include "logging.h" +#include "failtest_helper.h" + +int main(int argc, char *argv[]) +{ + unsigned int i, messages = 0; + struct tdb_context *tdb; + int flags[] = { TDB_DEFAULT, TDB_NOMMAP, + TDB_CONVERT, TDB_NOMMAP|TDB_CONVERT }; + + failtest_init(argc, argv); + failtest_hook = block_repeat_failures; + failtest_exit_check = exit_check_log; + plan_tests(sizeof(flags) / sizeof(flags[0]) * 4); + for (i = 0; i < sizeof(flags) / sizeof(flags[0]); i++) { + tdb = tdb_open("run-35-convert.tdb", flags[i], + O_RDWR|O_CREAT|O_TRUNC, 0600, &tap_log_attr); + if (!ok1(tdb)) + failtest_exit(exit_status()); + + tdb_close(tdb); + /* If we say TDB_CONVERT, it must be converted */ + tdb = tdb_open("run-35-convert.tdb", + flags[i]|TDB_CONVERT, + O_RDWR, 0600, &tap_log_attr); + if (flags[i] & TDB_CONVERT) { + if (!tdb) + failtest_exit(exit_status()); + ok1(tdb_get_flags(tdb) & TDB_CONVERT); + tdb_close(tdb); + } else { + if (!ok1(!tdb && errno == EIO)) + failtest_exit(exit_status()); + ok1(tap_log_messages == ++messages); + if (!ok1(log_last && strstr(log_last, "TDB_CONVERT"))) + failtest_exit(exit_status()); + } + + /* If don't say TDB_CONVERT, it *may* be converted */ + tdb = tdb_open("run-35-convert.tdb", + flags[i] & ~TDB_CONVERT, + O_RDWR, 0600, &tap_log_attr); + if (!tdb) + failtest_exit(exit_status()); + ok1(tdb_get_flags(tdb) == flags[i]); + tdb_close(tdb); + } + failtest_exit(exit_status()); +} |