diff options
| -rw-r--r-- | lib/tdb2/open.c | 7 | ||||
| -rw-r--r-- | lib/tdb2/test/api-tdb1-flag-removal.c | 38 |
2 files changed, 40 insertions, 5 deletions
diff --git a/lib/tdb2/open.c b/lib/tdb2/open.c index 18e4cd8bb0..b76bd24b2a 100644 --- a/lib/tdb2/open.c +++ b/lib/tdb2/open.c @@ -647,12 +647,9 @@ struct tdb_context *tdb_open(const char *name, int tdb_flags, goto fail; } + /* This is a version2 tdb. */ if (tdb->flags & TDB_VERSION1) { - ecode = tdb_logerr(tdb, TDB_ERR_IO, TDB_LOG_ERROR, - "tdb_open:" - " %s does not need TDB_VERSION1", - name); - goto fail; + tdb->flags &= ~TDB_VERSION1; } tdb2_context_init(tdb); diff --git a/lib/tdb2/test/api-tdb1-flag-removal.c b/lib/tdb2/test/api-tdb1-flag-removal.c new file mode 100644 index 0000000000..28f24e6388 --- /dev/null +++ b/lib/tdb2/test/api-tdb1-flag-removal.c @@ -0,0 +1,38 @@ +#include <ccan/tdb2/tdb2.h> +#include <ccan/tap/tap.h> +#include <ccan/hash/hash.h> +#include <sys/types.h> +#include <sys/stat.h> +#include <fcntl.h> + +#include "logging.h" + +int main(int argc, char *argv[]) +{ + unsigned int i; + struct tdb_context *tdb; + int flags[] = { TDB_DEFAULT, TDB_NOMMAP, + TDB_CONVERT, TDB_NOMMAP|TDB_CONVERT }; + + plan_tests(sizeof(flags) / sizeof(flags[0]) * 3 + 1); + for (i = 0; i < sizeof(flags) / sizeof(flags[0]); i++) { + tdb = tdb_open("run-12-store.tdb", flags[i], + O_RDWR|O_CREAT|O_TRUNC, 0600, &tap_log_attr); + if (!ok1(tdb)) + continue; + + tdb_close(tdb); + + tdb = tdb_open("run-12-store.tdb", flags[i] | TDB_VERSION1, + O_RDWR, 0600, &tap_log_attr); + if (!ok1(tdb)) + continue; + /* It's not a version1 */ + ok1(!(tdb_get_flags(tdb) & TDB_VERSION1)); + + tdb_close(tdb); + } + + ok1(tap_log_messages == 0); + return exit_status(); +} |