summaryrefslogtreecommitdiff
path: root/source4/torture/local/dbspeed.c
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>2007-02-07 02:57:44 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 14:44:45 -0500
commite5b00e3637b82bc72382e1a5cd75a521c7598743 (patch)
tree94169bdd75b8a326548d445f5252848c74681160 /source4/torture/local/dbspeed.c
parentb115da2230944aa13ea777ccc65bf978e257e4cb (diff)
downloadsamba-e5b00e3637b82bc72382e1a5cd75a521c7598743.tar.gz
samba-e5b00e3637b82bc72382e1a5cd75a521c7598743.tar.bz2
samba-e5b00e3637b82bc72382e1a5cd75a521c7598743.zip
r21211: fixed 3 places that assumed that torture_fail() didn't call
return. Having a lowercase macro that returns from a function is probably a bad idea - it looks too much like a C function but doesn't behave like one. (This used to be commit e680656419f00fd8b4ddb996fd247bada0f0a51d)
Diffstat (limited to 'source4/torture/local/dbspeed.c')
-rw-r--r--source4/torture/local/dbspeed.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/source4/torture/local/dbspeed.c b/source4/torture/local/dbspeed.c
index 298c5e3a2a..ba2517e24d 100644
--- a/source4/torture/local/dbspeed.c
+++ b/source4/torture/local/dbspeed.c
@@ -63,8 +63,9 @@ static BOOL test_tdb_speed(struct torture_context *torture, const void *_data)
tdbw = tdb_wrap_open(tmp_ctx, "test.tdb",
10000, 0, O_RDWR|O_CREAT|O_TRUNC, 0600);
if (!tdbw) {
+ unlink("test.tdb");
+ talloc_free(tmp_ctx);
torture_fail(torture, "Failed to open test.tdb");
- goto failed;
}
torture_comment(torture, "Adding %d SID records\n", torture_entries);
@@ -171,8 +172,9 @@ static BOOL test_ldb_speed(struct torture_context *torture, const void *_data)
ldb = ldb_wrap_connect(tmp_ctx, "tdb://test.ldb",
NULL, NULL, LDB_FLG_NOSYNC, NULL);
if (!ldb) {
+ unlink("./test.ldb");
+ talloc_free(tmp_ctx);
torture_fail(torture, "Failed to open test.ldb");
- goto failed;
}
/* add an index */
@@ -220,8 +222,9 @@ static BOOL test_ldb_speed(struct torture_context *torture, const void *_data)
}
if (talloc_total_blocks(torture) > 100) {
+ unlink("./test.ldb");
+ talloc_free(tmp_ctx);
torture_fail(torture, "memory leak in ldb search");
- goto failed;
}
torture_comment(torture, "ldb speed %.2f ops/sec\n", count/timeval_elapsed(&tv));