summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRusty Russell <rusty@rustcorp.com.au>2012-06-22 09:44:41 +0930
committerRusty Russell <rusty@rustcorp.com.au>2012-06-22 07:35:17 +0200
commit4c51ee1116c983ceef3804d79954aafad1b935ad (patch)
tree05e4806dcf3310315b489ff15e937c6e3f9ad7a5
parent01ec4a72de56ade54bbbc92e0a408771390c5c12 (diff)
downloadsamba-4c51ee1116c983ceef3804d79954aafad1b935ad.tar.gz
samba-4c51ee1116c983ceef3804d79954aafad1b935ad.tar.bz2
samba-4c51ee1116c983ceef3804d79954aafad1b935ad.zip
ntdb: test arbitrary operations during ntdb_parse_record().
In particular, this tests that we can store enough records to make the database expand while we map the given record. We use a global lock for this, but it could happen in theory with another process. It also tests the that we can recurse inside ntdb_parse_record(). Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
-rw-r--r--lib/ntdb/test/api-94-expand-during-parse.c89
-rw-r--r--lib/ntdb/wscript1
2 files changed, 90 insertions, 0 deletions
diff --git a/lib/ntdb/test/api-94-expand-during-parse.c b/lib/ntdb/test/api-94-expand-during-parse.c
new file mode 100644
index 0000000000..4963e47fe7
--- /dev/null
+++ b/lib/ntdb/test/api-94-expand-during-parse.c
@@ -0,0 +1,89 @@
+/* We use direct access to hand to the parse function: what if db expands? */
+#include "config.h"
+#include "ntdb.h"
+#include "tap-interface.h"
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <fcntl.h>
+#include "logging.h"
+#include "../private.h" /* To establish size, esp. for NTDB_INTERNAL dbs */
+
+static struct ntdb_context *ntdb;
+
+static off_t ntdb_size(void)
+{
+ return ntdb->file->map_size;
+}
+
+struct parse_info {
+ unsigned int depth;
+ NTDB_DATA expected;
+};
+
+static enum NTDB_ERROR parse(NTDB_DATA key, NTDB_DATA data,
+ struct parse_info *pinfo)
+{
+ off_t flen;
+ unsigned int i;
+
+ if (!ntdb_deq(data, pinfo->expected))
+ return NTDB_ERR_EINVAL;
+
+ flen = ntdb_size();
+
+ for (i = 0; ntdb_size() == flen; i++) {
+ NTDB_DATA add = ntdb_mkdata(&i, sizeof(i));
+
+ /* This is technically illegal parse(), which is why we
+ * grabbed allrecord lock.*/
+ ntdb_store(ntdb, add, add, NTDB_INSERT);
+ }
+
+ /* Access the record again. */
+ if (!ntdb_deq(data, pinfo->expected))
+ return NTDB_ERR_EINVAL;
+
+ /* Recurse! Woot! */
+ if (pinfo->depth != 0) {
+ enum NTDB_ERROR ecode;
+
+ pinfo->depth--;
+ ecode = ntdb_parse_record(ntdb, key, parse, pinfo);
+ if (ecode) {
+ return ecode;
+ }
+ }
+
+ /* Access the record one more time. */
+ if (!ntdb_deq(data, pinfo->expected))
+ return NTDB_ERR_EINVAL;
+
+ return NTDB_SUCCESS;
+}
+
+int main(int argc, char *argv[])
+{
+ unsigned int i;
+ int flags[] = { NTDB_INTERNAL, NTDB_DEFAULT, NTDB_NOMMAP,
+ NTDB_INTERNAL|NTDB_CONVERT, NTDB_CONVERT,
+ NTDB_NOMMAP|NTDB_CONVERT };
+ struct parse_info pinfo;
+ NTDB_DATA key = ntdb_mkdata("hello", 5), data = ntdb_mkdata("world", 5);
+
+ plan_tests(sizeof(flags) / sizeof(flags[0]) * 3 + 1);
+ for (i = 0; i < sizeof(flags) / sizeof(flags[0]); i++) {
+ ntdb = ntdb_open("api-94-expand-during-parse.ntdb",
+ flags[i]|MAYBE_NOSYNC,
+ O_RDWR|O_CREAT|O_TRUNC, 0600, &tap_log_attr);
+ ok1(ntdb_store(ntdb, key, data, NTDB_INSERT) == NTDB_SUCCESS);
+ ok1(ntdb_lockall(ntdb) == NTDB_SUCCESS);
+ pinfo.expected = data;
+ pinfo.depth = 3;
+ ok1(ntdb_parse_record(ntdb, key, parse, &pinfo) == NTDB_SUCCESS);
+ ntdb_unlockall(ntdb);
+ ntdb_close(ntdb);
+ }
+
+ ok1(tap_log_messages == 0);
+ return exit_status();
+}
diff --git a/lib/ntdb/wscript b/lib/ntdb/wscript
index e211a9fc0c..abf6db3ed0 100644
--- a/lib/ntdb/wscript
+++ b/lib/ntdb/wscript
@@ -79,6 +79,7 @@ def configure(conf):
'test/api-91-get-stats.c',
'test/api-92-get-set-readonly.c',
'test/api-93-repack.c',
+ 'test/api-94-expand-during-parse.c',
'test/api-95-read-only-during-parse.c',
'test/api-add-remove-flags.c',
'test/api-check-callback.c',