r26644: Janitorial: Pass resolve_context explicitly to various SMB functions, should...
[jelmer/samba4-debian.git] / source / torture / basic / base.c
index 29f0996bd458429b645f7718690fbeb19d7477c7..9820a02f2746fb968c291f32bd9d92a162e1ec5b 100644 (file)
@@ -6,7 +6,7 @@
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 2 of the License, or
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
    
    This program is distributed in the hope that it will be useful,
@@ -15,8 +15,7 @@
    GNU General Public License for more details.
    
    You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "includes.h"
 #include "librpc/gen_ndr/ndr_nbt.h"
 #include "lib/events/events.h"
 #include "lib/cmdline/popt_common.h"
+#include "param/param.h"
 
 
 #define CHECK_MAX_FAILURES(label) do { if (++failures >= torture_failures) goto label; } while (0)
 
 
-static struct smbcli_state *open_nbt_connection(void)
+static struct smbcli_state *open_nbt_connection(struct torture_context *tctx)
 {
        struct nbt_name called, calling;
        struct smbcli_state *cli;
-       const char *host = lp_parm_string(-1, "torture", "host");
+       const char *host = torture_setting_string(tctx, "host", NULL);
 
-       make_nbt_name_client(&calling, lp_netbios_name());
+       make_nbt_name_client(&calling, lp_netbios_name(tctx->lp_ctx));
 
        nbt_choose_called_name(NULL, &called, host, NBT_NAME_SERVER);
 
        cli = smbcli_state_init(NULL);
        if (!cli) {
-               printf("Failed initialize smbcli_struct to connect with %s\n", host);
+               torture_comment(tctx, "Failed initialize smbcli_struct to connect with %s\n", host);
                goto failed;
        }
 
-       if (!smbcli_socket_connect(cli, host)) {
-               printf("Failed to connect with %s\n", host);
+       if (!smbcli_socket_connect(cli, host, lp_smb_ports(tctx->lp_ctx), lp_resolve_context(tctx->lp_ctx), lp_max_xmit(tctx->lp_ctx), lp_maxmux(tctx->lp_ctx), lp_nt_status_support(tctx->lp_ctx) && lp_use_spnego(tctx->lp_ctx))) {
+               torture_comment(tctx, "Failed to connect with %s\n", host);
                goto failed;
        }
 
        if (!smbcli_transport_establish(cli, &calling, &called)) {
-               printf("%s rejected the session\n",host);
+               torture_comment(tctx, "%s rejected the session\n",host);
                goto failed;
        }
 
@@ -68,46 +68,42 @@ failed:
        return NULL;
 }
 
-static BOOL tcon_devtest(struct smbcli_state *cli,
-                        const char *myshare, const char *devtype,
-                        NTSTATUS expected_error)
+static bool tcon_devtest(struct torture_context *tctx, 
+                                                struct smbcli_state *cli,
+                                                const char *myshare, const char *devtype,
+                                                NTSTATUS expected_error)
 {
-       BOOL status;
-       BOOL ret;
-       const char *password = lp_parm_string(-1, "torture", "password");
+       bool status;
+       const char *password = torture_setting_string(tctx, "password", NULL);
 
        status = NT_STATUS_IS_OK(smbcli_tconX(cli, myshare, devtype, 
                                                password));
 
-       printf("Trying share %s with devtype %s\n", myshare, devtype);
+       torture_comment(tctx, "Trying share %s with devtype %s\n", myshare, devtype);
 
        if (NT_STATUS_IS_OK(expected_error)) {
-               if (status) {
-                       ret = True;
-               } else {
-                       printf("tconX to share %s with type %s "
-                              "should have succeeded but failed\n",
-                              myshare, devtype);
-                       ret = False;
+               if (!status) {
+                       torture_fail(tctx, talloc_asprintf(tctx, 
+                                  "tconX to share %s with type %s "
+                              "should have succeeded but failed",
+                              myshare, devtype));
                }
                smbcli_tdis(cli);
        } else {
                if (status) {
-                       printf("tconx to share %s with type %s "
-                              "should have failed but succeeded\n",
-                              myshare, devtype);
-                       ret = False;
+                       torture_fail(tctx, talloc_asprintf(tctx, 
+                                  "tconx to share %s with type %s "
+                              "should have failed but succeeded",
+                              myshare, devtype));
                } else {
                        if (NT_STATUS_EQUAL(smbcli_nt_error(cli->tree),
                                            expected_error)) {
-                               ret = True;
                        } else {
-                               printf("Returned unexpected error\n");
-                               ret = False;
+                               torture_fail(tctx, "Returned unexpected error");
                        }
                }
        }
-       return ret;
+       return true;
 }
 
 
@@ -116,77 +112,59 @@ static BOOL tcon_devtest(struct smbcli_state *cli,
 test whether fnums and tids open on one VC are available on another (a major
 security hole)
 */
-static BOOL run_fdpasstest(struct torture_context *torture)
+static bool run_fdpasstest(struct torture_context *tctx,
+                                                  struct smbcli_state *cli1, 
+                                                  struct smbcli_state *cli2)
 {
-       struct smbcli_state *cli1, *cli2;
        const char *fname = "\\fdpass.tst";
        int fnum1, oldtid;
        uint8_t buf[1024];
 
-       if (!torture_open_connection(&cli1) || !torture_open_connection(&cli2)) {
-               return False;
-       }
-
-       printf("starting fdpasstest\n");
-
        smbcli_unlink(cli1->tree, fname);
 
-       printf("Opening a file on connection 1\n");
+       torture_comment(tctx, "Opening a file on connection 1\n");
 
        fnum1 = smbcli_open(cli1->tree, fname, O_RDWR|O_CREAT|O_EXCL, DENY_NONE);
-       if (fnum1 == -1) {
-               printf("open of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
-       }
+       torture_assert(tctx, fnum1 != -1, 
+                       talloc_asprintf(tctx, 
+                       "open of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree)));
 
-       printf("writing to file on connection 1\n");
+       torture_comment(tctx, "writing to file on connection 1\n");
 
-       if (smbcli_write(cli1->tree, fnum1, 0, "hello world\n", 0, 13) != 13) {
-               printf("write failed (%s)\n", smbcli_errstr(cli1->tree));
-               return False;
-       }
+       torture_assert(tctx, 
+               smbcli_write(cli1->tree, fnum1, 0, "hello world\n", 0, 13) == 13,
+               talloc_asprintf(tctx, 
+               "write failed (%s)\n", smbcli_errstr(cli1->tree)));
 
        oldtid = cli2->tree->tid;
        cli2->session->vuid = cli1->session->vuid;
        cli2->tree->tid = cli1->tree->tid;
        cli2->session->pid = cli1->session->pid;
 
-       printf("reading from file on connection 2\n");
+       torture_comment(tctx, "reading from file on connection 2\n");
 
-       if (smbcli_read(cli2->tree, fnum1, buf, 0, 13) == 13) {
-               printf("read succeeded! nasty security hole [%s]\n",
-                      buf);
-               return False;
-       }
+       torture_assert(tctx, smbcli_read(cli2->tree, fnum1, buf, 0, 13) != 13,
+                                  talloc_asprintf(tctx,
+               "read succeeded! nasty security hole [%s]\n", buf));
 
        smbcli_close(cli1->tree, fnum1);
        smbcli_unlink(cli1->tree, fname);
 
        cli2->tree->tid = oldtid;
 
-       torture_close_connection(cli1);
-       torture_close_connection(cli2);
-
-       printf("finished fdpasstest\n");
-       return True;
+       return true;
 }
 
 /**
   This checks how the getatr calls works
 */
-static BOOL run_attrtest(struct torture_context *torture)
+static bool run_attrtest(struct torture_context *tctx, 
+                                                struct smbcli_state *cli)
 {
-       struct smbcli_state *cli;
        int fnum;
        time_t t, t2;
        const char *fname = "\\attrib123456789.tst";
-       BOOL correct = True;
-
-       printf("starting attrib test\n");
-
-       if (!torture_open_connection(&cli)) {
-               return False;
-       }
+       bool correct = true;
 
        smbcli_unlink(cli->tree, fname);
        fnum = smbcli_open(cli->tree, fname, 
@@ -194,59 +172,53 @@ static BOOL run_attrtest(struct torture_context *torture)
        smbcli_close(cli->tree, fnum);
 
        if (NT_STATUS_IS_ERR(smbcli_getatr(cli->tree, fname, NULL, NULL, &t))) {
-               printf("getatr failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
+               torture_comment(tctx, "getatr failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = false;
        }
 
-       printf("New file time is %s", ctime(&t));
+       torture_comment(tctx, "New file time is %s", ctime(&t));
 
        if (abs(t - time(NULL)) > 60*60*24*10) {
-               printf("ERROR: SMBgetatr bug. time is %s",
+               torture_comment(tctx, "ERROR: SMBgetatr bug. time is %s",
                       ctime(&t));
                t = time(NULL);
-               correct = False;
+               correct = false;
        }
 
        t2 = t-60*60*24; /* 1 day ago */
 
-       printf("Setting file time to %s", ctime(&t2));
+       torture_comment(tctx, "Setting file time to %s", ctime(&t2));
 
        if (NT_STATUS_IS_ERR(smbcli_setatr(cli->tree, fname, 0, t2))) {
-               printf("setatr failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = True;
+               torture_comment(tctx, "setatr failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = true;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_getatr(cli->tree, fname, NULL, NULL, &t))) {
-               printf("getatr failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = True;
+               torture_comment(tctx, "getatr failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = true;
        }
 
-       printf("Retrieved file time as %s", ctime(&t));
+       torture_comment(tctx, "Retrieved file time as %s", ctime(&t));
 
        if (t != t2) {
-               printf("ERROR: getatr/setatr bug. times are\n%s",
+               torture_comment(tctx, "ERROR: getatr/setatr bug. times are\n%s",
                       ctime(&t));
-               printf("%s", ctime(&t2));
-               correct = True;
+               torture_comment(tctx, "%s", ctime(&t2));
+               correct = true;
        }
 
        smbcli_unlink(cli->tree, fname);
 
-       if (!torture_close_connection(cli)) {
-               correct = False;
-       }
-
-       printf("attrib test finished\n");
-
        return correct;
 }
 
 /**
   This checks a couple of trans2 calls
 */
-static BOOL run_trans2test(struct torture_context *torture)
+static bool run_trans2test(struct torture_context *tctx, 
+                                                  struct smbcli_state *cli)
 {
-       struct smbcli_state *cli;
        int fnum;
        size_t size;
        time_t c_time, a_time, m_time, w_time, m_time2;
@@ -254,37 +226,31 @@ static BOOL run_trans2test(struct torture_context *torture)
        const char *dname = "\\trans2";
        const char *fname2 = "\\trans2\\trans2.tst";
        const char *pname;
-       BOOL correct = True;
-
-       printf("starting trans2 test\n");
-
-       if (!torture_open_connection(&cli)) {
-               return False;
-       }
+       bool correct = true;
 
        smbcli_unlink(cli->tree, fname);
 
-       printf("Testing qfileinfo\n");
+       torture_comment(tctx, "Testing qfileinfo\n");
        
        fnum = smbcli_open(cli->tree, fname, 
                        O_RDWR | O_CREAT | O_TRUNC, DENY_NONE);
        if (NT_STATUS_IS_ERR(smbcli_qfileinfo(cli->tree, fnum, NULL, &size, &c_time, &a_time, &m_time,
                           NULL, NULL))) {
-               printf("ERROR: qfileinfo failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
+               torture_comment(tctx, "ERROR: qfileinfo failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = false;
        }
 
-       printf("Testing NAME_INFO\n");
+       torture_comment(tctx, "Testing NAME_INFO\n");
 
        if (NT_STATUS_IS_ERR(smbcli_qfilename(cli->tree, fnum, &pname))) {
-               printf("ERROR: qfilename failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
+               torture_comment(tctx, "ERROR: qfilename failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = false;
        }
 
        if (!pname || strcmp(pname, fname)) {
-               printf("qfilename gave different name? [%s] [%s]\n",
+               torture_comment(tctx, "qfilename gave different name? [%s] [%s]\n",
                       fname, pname);
-               correct = False;
+               correct = false;
        }
 
        smbcli_close(cli->tree, fnum);
@@ -293,31 +259,31 @@ static BOOL run_trans2test(struct torture_context *torture)
        fnum = smbcli_open(cli->tree, fname, 
                        O_RDWR | O_CREAT | O_TRUNC, DENY_NONE);
        if (fnum == -1) {
-               printf("open of %s failed (%s)\n", fname, smbcli_errstr(cli->tree));
-               return False;
+               torture_comment(tctx, "open of %s failed (%s)\n", fname, smbcli_errstr(cli->tree));
+               return false;
        }
        smbcli_close(cli->tree, fnum);
 
-       printf("Checking for sticky create times\n");
+       torture_comment(tctx, "Checking for sticky create times\n");
 
        if (NT_STATUS_IS_ERR(smbcli_qpathinfo(cli->tree, fname, &c_time, &a_time, &m_time, &size, NULL))) {
-               printf("ERROR: qpathinfo failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
+               torture_comment(tctx, "ERROR: qpathinfo failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = false;
        } else {
                if (c_time != m_time) {
-                       printf("create time=%s", ctime(&c_time));
-                       printf("modify time=%s", ctime(&m_time));
-                       printf("This system appears to have sticky create times\n");
+                       torture_comment(tctx, "create time=%s", ctime(&c_time));
+                       torture_comment(tctx, "modify time=%s", ctime(&m_time));
+                       torture_comment(tctx, "This system appears to have sticky create times\n");
                }
                if (a_time % (60*60) == 0) {
-                       printf("access time=%s", ctime(&a_time));
-                       printf("This system appears to set a midnight access time\n");
-                       correct = False;
+                       torture_comment(tctx, "access time=%s", ctime(&a_time));
+                       torture_comment(tctx, "This system appears to set a midnight access time\n");
+                       correct = false;
                }
 
                if (abs(m_time - time(NULL)) > 60*60*24*7) {
-                       printf("ERROR: totally incorrect times - maybe word reversed? mtime=%s", ctime(&m_time));
-                       correct = False;
+                       torture_comment(tctx, "ERROR: totally incorrect times - maybe word reversed? mtime=%s", ctime(&m_time));
+                       correct = false;
                }
        }
 
@@ -327,13 +293,13 @@ static BOOL run_trans2test(struct torture_context *torture)
                        O_RDWR | O_CREAT | O_TRUNC, DENY_NONE);
        smbcli_close(cli->tree, fnum);
        if (NT_STATUS_IS_ERR(smbcli_qpathinfo2(cli->tree, fname, &c_time, &a_time, &m_time, &w_time, &size, NULL, NULL))) {
-               printf("ERROR: qpathinfo2 failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
+               torture_comment(tctx, "ERROR: qpathinfo2 failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = false;
        } else {
                if (w_time < 60*60*24*2) {
-                       printf("write time=%s", ctime(&w_time));
-                       printf("This system appears to set a initial 0 write time\n");
-                       correct = False;
+                       torture_comment(tctx, "write time=%s", ctime(&w_time));
+                       torture_comment(tctx, "This system appears to set a initial 0 write time\n");
+                       correct = false;
                }
        }
 
@@ -343,13 +309,13 @@ static BOOL run_trans2test(struct torture_context *torture)
        /* check if the server updates the directory modification time
            when creating a new file */
        if (NT_STATUS_IS_ERR(smbcli_mkdir(cli->tree, dname))) {
-               printf("ERROR: mkdir failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
+               torture_comment(tctx, "ERROR: mkdir failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = false;
        }
        sleep(3);
        if (NT_STATUS_IS_ERR(smbcli_qpathinfo2(cli->tree, "\\trans2\\", &c_time, &a_time, &m_time, &w_time, &size, NULL, NULL))) {
-               printf("ERROR: qpathinfo2 failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
+               torture_comment(tctx, "ERROR: qpathinfo2 failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = false;
        }
 
        fnum = smbcli_open(cli->tree, fname2, 
@@ -357,73 +323,65 @@ static BOOL run_trans2test(struct torture_context *torture)
        smbcli_write(cli->tree, fnum,  0, &fnum, 0, sizeof(fnum));
        smbcli_close(cli->tree, fnum);
        if (NT_STATUS_IS_ERR(smbcli_qpathinfo2(cli->tree, "\\trans2\\", &c_time, &a_time, &m_time2, &w_time, &size, NULL, NULL))) {
-               printf("ERROR: qpathinfo2 failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
+               torture_comment(tctx, "ERROR: qpathinfo2 failed (%s)\n", smbcli_errstr(cli->tree));
+               correct = false;
        } else {
                if (m_time2 == m_time) {
-                       printf("This system does not update directory modification times\n");
-                       correct = False;
+                       torture_comment(tctx, "This system does not update directory modification times\n");
+                       correct = false;
                }
        }
        smbcli_unlink(cli->tree, fname2);
        smbcli_rmdir(cli->tree, dname);
 
-       if (!torture_close_connection(cli)) {
-               correct = False;
-       }
-
-       printf("trans2 test finished\n");
-
        return correct;
 }
 
 /* send smb negprot commands, not reading the response */
-static BOOL run_negprot_nowait(struct torture_context *torture)
+static bool run_negprot_nowait(struct torture_context *tctx)
 {
        int i;
        struct smbcli_state *cli, *cli2;
-       BOOL correct = True;
+       bool correct = true;
 
-       printf("starting negprot nowait test\n");
+       torture_comment(tctx, "starting negprot nowait test\n");
 
-       cli = open_nbt_connection();
+       cli = open_nbt_connection(tctx);
        if (!cli) {
-               return False;
+               return false;
        }
 
-       printf("Filling send buffer\n");
+       torture_comment(tctx, "Filling send buffer\n");
 
        for (i=0;i<100;i++) {
                struct smbcli_request *req;
-               req = smb_raw_negotiate_send(cli->transport, PROTOCOL_NT1);
+               req = smb_raw_negotiate_send(cli->transport, lp_unicode(tctx->lp_ctx), PROTOCOL_NT1);
                event_loop_once(cli->transport->socket->event.ctx);
                if (req->state == SMBCLI_REQUEST_ERROR) {
                        if (i > 0) {
-                               printf("Failed to fill pipe packet[%d] - %s (ignored)\n", i+1, nt_errstr(req->status));
+                               torture_comment(tctx, "Failed to fill pipe packet[%d] - %s (ignored)\n", i+1, nt_errstr(req->status));
                                break;
                        } else {
-                               printf("Failed to fill pipe - %s \n", nt_errstr(req->status));
+                               torture_comment(tctx, "Failed to fill pipe - %s \n", nt_errstr(req->status));
                                torture_close_connection(cli);
-                               return False;
+                               return false;
                        }
                }
        }
 
-       printf("Opening secondary connection\n");
-       if (!torture_open_connection(&cli2)) {
-               printf("Failed to open secondary connection\n");
-               correct = False;
+       torture_comment(tctx, "Opening secondary connection\n");
+       if (!torture_open_connection(&cli2, tctx, 1)) {
+               torture_comment(tctx, "Failed to open secondary connection\n");
+               correct = false;
        }
 
        if (!torture_close_connection(cli2)) {
-               printf("Failed to close secondary connection\n");
-               correct = False;
+               torture_comment(tctx, "Failed to close secondary connection\n");
+               correct = false;
        }
 
        torture_close_connection(cli);
 
-       printf("finished negprot nowait test\n");
-
        return correct;
 }
 
@@ -431,51 +389,44 @@ static BOOL run_negprot_nowait(struct torture_context *torture)
   this checks to see if a secondary tconx can use open files from an
   earlier tconx
  */
-static BOOL run_tcon_test(struct torture_context *torture)
+static bool run_tcon_test(struct torture_context *tctx, struct smbcli_state *cli)
 {
-       struct smbcli_state *cli;
        const char *fname = "\\tcontest.tmp";
        int fnum1;
        uint16_t cnum1, cnum2, cnum3;
        uint16_t vuid1, vuid2;
        uint8_t buf[4];
-       BOOL ret = True;
+       bool ret = true;
        struct smbcli_tree *tree1;
-       const char *host = lp_parm_string(-1, "torture", "host");
-       const char *share = lp_parm_string(-1, "torture", "share");
-       const char *password = lp_parm_string(-1, "torture", "password");
-
-       if (!torture_open_connection(&cli)) {
-               return False;
-       }
-
-       printf("starting tcontest\n");
+       const char *host = torture_setting_string(tctx, "host", NULL);
+       const char *share = torture_setting_string(tctx, "share", NULL);
+       const char *password = torture_setting_string(tctx, "password", NULL);
 
        if (smbcli_deltree(cli->tree, fname) == -1) {
-               printf("unlink of %s failed (%s)\n", fname, smbcli_errstr(cli->tree));
+               torture_comment(tctx, "unlink of %s failed (%s)\n", fname, smbcli_errstr(cli->tree));
        }
 
        fnum1 = smbcli_open(cli->tree, fname, O_RDWR|O_CREAT|O_EXCL, DENY_NONE);
        if (fnum1 == -1) {
-               printf("open of %s failed (%s)\n", fname, smbcli_errstr(cli->tree));
-               return False;
+               torture_comment(tctx, "open of %s failed (%s)\n", fname, smbcli_errstr(cli->tree));
+               return false;
        }
 
        cnum1 = cli->tree->tid;
        vuid1 = cli->session->vuid;
 
-       memset(&buf, 0, 4); /* init buf so valgrind won't complain */
+       memset(buf, 0, 4); /* init buf so valgrind won't complain */
        if (smbcli_write(cli->tree, fnum1, 0, buf, 130, 4) != 4) {
-               printf("initial write failed (%s)\n", smbcli_errstr(cli->tree));
-               return False;
+               torture_comment(tctx, "initial write failed (%s)\n", smbcli_errstr(cli->tree));
+               return false;
        }
 
        tree1 = cli->tree;      /* save old tree connection */
        if (NT_STATUS_IS_ERR(smbcli_tconX(cli, share, "?????", password))) {
-               printf("%s refused 2nd tree connect (%s)\n", host,
+               torture_comment(tctx, "%s refused 2nd tree connect (%s)\n", host,
                           smbcli_errstr(cli->tree));
                talloc_free(cli);
-               return False;
+               return false;
        }
 
        cnum2 = cli->tree->tid;
@@ -486,10 +437,10 @@ static BOOL run_tcon_test(struct torture_context *torture)
        cli->tree->tid = cnum2;
 
        if (smbcli_write(cli->tree, fnum1, 0, buf, 130, 4) == 4) {
-               printf("* server allows write with wrong TID\n");
-               ret = False;
+               torture_comment(tctx, "* server allows write with wrong TID\n");
+               ret = false;
        } else {
-               printf("server fails write with wrong TID : %s\n", smbcli_errstr(cli->tree));
+               torture_comment(tctx, "server fails write with wrong TID : %s\n", smbcli_errstr(cli->tree));
        }
 
 
@@ -497,10 +448,10 @@ static BOOL run_tcon_test(struct torture_context *torture)
        cli->tree->tid = cnum3;
 
        if (smbcli_write(cli->tree, fnum1, 0, buf, 130, 4) == 4) {
-               printf("* server allows write with invalid TID\n");
-               ret = False;
+               torture_comment(tctx, "* server allows write with invalid TID\n");
+               ret = false;
        } else {
-               printf("server fails write with invalid TID : %s\n", smbcli_errstr(cli->tree));
+               torture_comment(tctx, "server fails write with invalid TID : %s\n", smbcli_errstr(cli->tree));
        }
 
        /* try a write with an invalid vuid */
@@ -508,25 +459,25 @@ static BOOL run_tcon_test(struct torture_context *torture)
        cli->tree->tid = cnum1;
 
        if (smbcli_write(cli->tree, fnum1, 0, buf, 130, 4) == 4) {
-               printf("* server allows write with invalid VUID\n");
-               ret = False;
+               torture_comment(tctx, "* server allows write with invalid VUID\n");
+               ret = false;
        } else {
-               printf("server fails write with invalid VUID : %s\n", smbcli_errstr(cli->tree));
+               torture_comment(tctx, "server fails write with invalid VUID : %s\n", smbcli_errstr(cli->tree));
        }
 
        cli->session->vuid = vuid1;
        cli->tree->tid = cnum1;
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli->tree, fnum1))) {
-               printf("close failed (%s)\n", smbcli_errstr(cli->tree));
-               return False;
+               torture_comment(tctx, "close failed (%s)\n", smbcli_errstr(cli->tree));
+               return false;
        }
 
        cli->tree->tid = cnum2;
 
        if (NT_STATUS_IS_ERR(smbcli_tdis(cli))) {
-               printf("secondary tdis failed (%s)\n", smbcli_errstr(cli->tree));
-               return False;
+               torture_comment(tctx, "secondary tdis failed (%s)\n", smbcli_errstr(cli->tree));
+               return false;
        }
 
        cli->tree = tree1;  /* restore initial tree */
@@ -534,73 +485,52 @@ static BOOL run_tcon_test(struct torture_context *torture)
 
        smbcli_unlink(tree1, fname);
 
-       if (!torture_close_connection(cli)) {
-               return False;
-       }
-
        return ret;
 }
 
 /**
  checks for correct tconX support
  */
-static BOOL run_tcon_devtype_test(struct torture_context *torture)
+static bool run_tcon_devtype_test(struct torture_context *tctx, 
+                                                                 struct smbcli_state *cli1)
 {
-       struct smbcli_state *cli1 = NULL;
-       NTSTATUS status;
-       BOOL ret = True;
-       const char *host = lp_parm_string(-1, "torture", "host");
-       const char *share = lp_parm_string(-1, "torture", "share");
-       
-       status = smbcli_full_connection(NULL,
-                                       &cli1, host, 
-                                       share, NULL,
-                                       cmdline_credentials, NULL);
-
-       if (!NT_STATUS_IS_OK(status)) {
-               printf("could not open connection\n");
-               return False;
-       }
+       const char *share = torture_setting_string(tctx, "share", NULL);
 
-       if (!tcon_devtest(cli1, "IPC$", "A:", NT_STATUS_BAD_DEVICE_TYPE))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, "IPC$", "A:", NT_STATUS_BAD_DEVICE_TYPE))
+               return false;
 
-       if (!tcon_devtest(cli1, "IPC$", "?????", NT_STATUS_OK))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, "IPC$", "?????", NT_STATUS_OK))
+               return false;
 
-       if (!tcon_devtest(cli1, "IPC$", "LPT:", NT_STATUS_BAD_DEVICE_TYPE))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, "IPC$", "LPT:", NT_STATUS_BAD_DEVICE_TYPE))
+               return false;
 
-       if (!tcon_devtest(cli1, "IPC$", "IPC", NT_STATUS_OK))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, "IPC$", "IPC", NT_STATUS_OK))
+               return false;
                        
-       if (!tcon_devtest(cli1, "IPC$", "FOOBA", NT_STATUS_BAD_DEVICE_TYPE))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, "IPC$", "FOOBA", NT_STATUS_BAD_DEVICE_TYPE))
+               return false;
 
-       if (!tcon_devtest(cli1, share, "A:", NT_STATUS_OK))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, share, "A:", NT_STATUS_OK))
+               return false;
 
-       if (!tcon_devtest(cli1, share, "?????", NT_STATUS_OK))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, share, "?????", NT_STATUS_OK))
+               return false;
 
-       if (!tcon_devtest(cli1, share, "LPT:", NT_STATUS_BAD_DEVICE_TYPE))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, share, "LPT:", NT_STATUS_BAD_DEVICE_TYPE))
+               return false;
 
-       if (!tcon_devtest(cli1, share, "IPC", NT_STATUS_BAD_DEVICE_TYPE))
-               ret = False;
+       if (!tcon_devtest(tctx, cli1, share, "IPC", NT_STATUS_BAD_DEVICE_TYPE))
+               return false;
                        
-       if (!tcon_devtest(cli1, share, "FOOBA", NT_STATUS_BAD_DEVICE_TYPE))
-               ret = False;
-
-       talloc_free(cli1);
-
-       if (ret)
-               printf("Passed tcondevtest\n");
+       if (!tcon_devtest(tctx, cli1, share, "FOOBA", NT_STATUS_BAD_DEVICE_TYPE))
+               return false;
 
-       return ret;
+       return true;
 }
 
-static BOOL rw_torture2(struct smbcli_state *c1, struct smbcli_state *c2)
+static bool rw_torture2(struct torture_context *tctx,
+                                               struct smbcli_state *c1, struct smbcli_state *c2)
 {
        const char *lockfname = "\\torture2.lck";
        int fnum1;
@@ -608,133 +538,105 @@ static BOOL rw_torture2(struct smbcli_state *c1, struct smbcli_state *c2)
        int i;
        uint8_t buf[131072];
        uint8_t buf_rd[131072];
-       BOOL correct = True;
+       bool correct = true;
        ssize_t bytes_read, bytes_written;
 
-       if (smbcli_deltree(c1->tree, lockfname) == -1) {
-               printf("unlink failed (%s)\n", smbcli_errstr(c1->tree));
-       }
+       torture_assert(tctx, smbcli_deltree(c1->tree, lockfname) != -1,
+                                  talloc_asprintf(tctx, 
+               "unlink failed (%s)", smbcli_errstr(c1->tree)));
 
        fnum1 = smbcli_open(c1->tree, lockfname, O_RDWR | O_CREAT | O_EXCL, 
                         DENY_NONE);
-       if (fnum1 == -1) {
-               printf("first open read/write of %s failed (%s)\n",
-                               lockfname, smbcli_errstr(c1->tree));
-               return False;
-       }
+       torture_assert(tctx, fnum1 != -1, 
+                                  talloc_asprintf(tctx, 
+               "first open read/write of %s failed (%s)",
+                               lockfname, smbcli_errstr(c1->tree)));
        fnum2 = smbcli_open(c2->tree, lockfname, O_RDONLY, 
                         DENY_NONE);
-       if (fnum2 == -1) {
-               printf("second open read-only of %s failed (%s)\n",
-                               lockfname, smbcli_errstr(c2->tree));
-               smbcli_close(c1->tree, fnum1);
-               return False;
-       }
+       torture_assert(tctx, fnum2 != -1, 
+                                  talloc_asprintf(tctx, 
+               "second open read-only of %s failed (%s)",
+                               lockfname, smbcli_errstr(c2->tree)));
 
-       printf("Checking data integrity over %d ops\n", torture_numops);
+       torture_comment(tctx, "Checking data integrity over %d ops\n", 
+                                       torture_numops);
 
        for (i=0;i<torture_numops;i++)
        {
                size_t buf_size = ((uint_t)random()%(sizeof(buf)-1))+ 1;
                if (i % 10 == 0) {
-                       printf("%d\r", i); fflush(stdout);
+                       if (torture_setting_bool(tctx, "progress", true)) {
+                               torture_comment(tctx, "%d\r", i); fflush(stdout);
+                       }
                }
 
                generate_random_buffer(buf, buf_size);
 
                if ((bytes_written = smbcli_write(c1->tree, fnum1, 0, buf, 0, buf_size)) != buf_size) {
-                       printf("write failed (%s)\n", smbcli_errstr(c1->tree));
-                       printf("wrote %d, expected %d\n", (int)bytes_written, (int)buf_size); 
-                       correct = False;
+                       torture_comment(tctx, "write failed (%s)\n", smbcli_errstr(c1->tree));
+                       torture_comment(tctx, "wrote %d, expected %d\n", (int)bytes_written, (int)buf_size); 
+                       correct = false;
                        break;
                }
 
                if ((bytes_read = smbcli_read(c2->tree, fnum2, buf_rd, 0, buf_size)) != buf_size) {
-                       printf("read failed (%s)\n", smbcli_errstr(c2->tree));
-                       printf("read %d, expected %d\n", (int)bytes_read, (int)buf_size); 
-                       correct = False;
+                       torture_comment(tctx, "read failed (%s)\n", smbcli_errstr(c2->tree));
+                       torture_comment(tctx, "read %d, expected %d\n", (int)bytes_read, (int)buf_size); 
+                       correct = false;
                        break;
                }
 
-               if (memcmp(buf_rd, buf, buf_size) != 0)
-               {
-                       printf("read/write compare failed\n");
-                       correct = False;
-                       break;
-               }
+               torture_assert(tctx, memcmp(buf_rd, buf, buf_size) == 0, 
+                       "read/write compare failed\n");
        }
 
-       if (NT_STATUS_IS_ERR(smbcli_close(c2->tree, fnum2))) {
-               printf("close failed (%s)\n", smbcli_errstr(c2->tree));
-               correct = False;
-       }
-       if (NT_STATUS_IS_ERR(smbcli_close(c1->tree, fnum1))) {
-               printf("close failed (%s)\n", smbcli_errstr(c1->tree));
-               correct = False;
-       }
+       torture_assert_ntstatus_ok(tctx, smbcli_close(c2->tree, fnum2), 
+               talloc_asprintf(tctx, "close failed (%s)", smbcli_errstr(c2->tree)));
+       torture_assert_ntstatus_ok(tctx, smbcli_close(c1->tree, fnum1),
+               talloc_asprintf(tctx, "close failed (%s)", smbcli_errstr(c1->tree)));
 
-       if (NT_STATUS_IS_ERR(smbcli_unlink(c1->tree, lockfname))) {
-               printf("unlink failed (%s)\n", smbcli_errstr(c1->tree));
-               correct = False;
-       }
+       torture_assert_ntstatus_ok(tctx, smbcli_unlink(c1->tree, lockfname),
+               talloc_asprintf(tctx, "unlink failed (%s)", smbcli_errstr(c1->tree)));
+
+       torture_comment(tctx, "\n");
 
        return correct;
 }
 
 
 
-#define BOOLSTR(b) ((b) ? "Yes" : "No")
-
-static BOOL run_readwritetest(struct torture_context *torture)
+static bool run_readwritetest(struct torture_context *tctx,
+                                                         struct smbcli_state *cli1,
+                                                         struct smbcli_state *cli2)
 {
-       struct smbcli_state *cli1, *cli2;
-       BOOL test1, test2 = True;
-
-       if (!torture_open_connection(&cli1) || !torture_open_connection(&cli2)) {
-               return False;
-       }
-
-       printf("starting readwritetest\n");
+       torture_comment(tctx, "Running readwritetest v1\n");
+       if (!rw_torture2(tctx, cli1, cli2)) 
+               return false;
 
-       test1 = rw_torture2(cli1, cli2);
-       printf("Passed readwritetest v1: %s\n", BOOLSTR(test1));
+       torture_comment(tctx, "Running readwritetest v2\n");
 
-       if (test1) {
-               test2 = rw_torture2(cli1, cli1);
-               printf("Passed readwritetest v2: %s\n", BOOLSTR(test2));
-       }
-
-       if (!torture_close_connection(cli1)) {
-               test1 = False;
-       }
-
-       if (!torture_close_connection(cli2)) {
-               test2 = False;
-       }
+       if (!rw_torture2(tctx, cli1, cli1))
+               return false;
 
-       return (test1 && test2);
+       return true;
 }
 
-
-
-
-
 /*
 test the timing of deferred open requests
 */
-static BOOL run_deferopen(struct smbcli_state *cli, int dummy)
+static bool run_deferopen(struct torture_context *tctx, struct smbcli_state *cli, int dummy)
 {
        const char *fname = "\\defer_open_test.dat";
        int retries=4;
        int i = 0;
-       BOOL correct = True;
+       bool correct = true;
 
        if (retries <= 0) {
-               printf("failed to connect\n");
-               return False;
+               torture_comment(tctx, "failed to connect\n");
+               return false;
        }
 
-       printf("Testing deferred open requests.\n");
+       torture_comment(tctx, "Testing deferred open requests.\n");
 
        while (i < 4) {
                int fnum = -1;
@@ -753,24 +655,24 @@ static BOOL run_deferopen(struct smbcli_state *cli, int dummy)
                        if (NT_STATUS_EQUAL(smbcli_nt_error(cli->tree),NT_STATUS_SHARING_VIOLATION)) {
                                double e = timeval_elapsed(&tv);
                                if (e < 0.5 || e > 1.5) {
-                                       fprintf(stderr,"Timing incorrect %.2f violation\n",
+                                       torture_comment(tctx,"Timing incorrect %.2f violation\n",
                                                e);
                                }
                        }
                } while (NT_STATUS_EQUAL(smbcli_nt_error(cli->tree),NT_STATUS_SHARING_VIOLATION));
 
                if (fnum == -1) {
-                       fprintf(stderr,"Failed to open %s, error=%s\n", fname, smbcli_errstr(cli->tree));
-                       return False;
+                       torture_comment(tctx,"Failed to open %s, error=%s\n", fname, smbcli_errstr(cli->tree));
+                       return false;
                }
 
-               printf("pid %u open %d\n", getpid(), i);
+               torture_comment(tctx, "pid %u open %d\n", (unsigned)getpid(), i);
 
                sleep(10);
                i++;
                if (NT_STATUS_IS_ERR(smbcli_close(cli->tree, fnum))) {
-                       fprintf(stderr,"Failed to close %s, error=%s\n", fname, smbcli_errstr(cli->tree));
-                       return False;
+                       torture_comment(tctx,"Failed to close %s, error=%s\n", fname, smbcli_errstr(cli->tree));
+                       return false;
                }
                sleep(2);
        }
@@ -778,15 +680,12 @@ static BOOL run_deferopen(struct smbcli_state *cli, int dummy)
        if (NT_STATUS_IS_ERR(smbcli_unlink(cli->tree, fname))) {
                /* All until the last unlink will fail with sharing violation. */
                if (!NT_STATUS_EQUAL(smbcli_nt_error(cli->tree),NT_STATUS_SHARING_VIOLATION)) {
-                       printf("unlink of %s failed (%s)\n", fname, smbcli_errstr(cli->tree));
-                       correct = False;
+                       torture_comment(tctx, "unlink of %s failed (%s)\n", fname, smbcli_errstr(cli->tree));
+                       correct = false;
                }
        }
 
-       printf("deferred test finished\n");
-       if (!torture_close_connection(cli)) {
-               correct = False;
-       }
+       torture_comment(tctx, "deferred test finished\n");
        return correct;
 }
 
@@ -794,24 +693,17 @@ static BOOL run_deferopen(struct smbcli_state *cli, int dummy)
   Try with a wrong vuid and check error message.
  */
 
-static BOOL run_vuidtest(struct torture_context *torture)
+static bool run_vuidtest(struct torture_context *tctx, 
+                                                struct smbcli_state *cli)
 {
-       struct smbcli_state *cli;
        const char *fname = "\\vuid.tst";
        int fnum;
        size_t size;
        time_t c_time, a_time, m_time;
-       BOOL correct = True;
 
        uint16_t orig_vuid;
        NTSTATUS result;
 
-       printf("starting vuid test\n");
-
-       if (!torture_open_connection(&cli)) {
-               return False;
-       }
-
        smbcli_unlink(cli->tree, fname);
 
        fnum = smbcli_open(cli->tree, fname, 
@@ -821,66 +713,50 @@ static BOOL run_vuidtest(struct torture_context *torture)
 
        cli->session->vuid += 1234;
 
-       printf("Testing qfileinfo with wrong vuid\n");
+       torture_comment(tctx, "Testing qfileinfo with wrong vuid\n");
        
        if (NT_STATUS_IS_OK(result = smbcli_qfileinfo(cli->tree, fnum, NULL,
                                                   &size, &c_time, &a_time,
                                                   &m_time, NULL, NULL))) {
-               printf("ERROR: qfileinfo passed with wrong vuid\n");
-               correct = False;
+               torture_fail(tctx, "qfileinfo passed with wrong vuid");
        }
 
        if (!NT_STATUS_EQUAL(cli->transport->error.e.nt_status, 
                             NT_STATUS_DOS(ERRSRV, ERRbaduid)) &&
            !NT_STATUS_EQUAL(cli->transport->error.e.nt_status, 
                             NT_STATUS_INVALID_HANDLE)) {
-               printf("ERROR: qfileinfo should have returned DOS error "
-                      "ERRSRV:ERRbaduid\n  but returned %s\n",
-                      smbcli_errstr(cli->tree));
-               correct = False;
+               torture_fail(tctx, talloc_asprintf(tctx, 
+                               "qfileinfo should have returned DOS error "
+                      "ERRSRV:ERRbaduid\n  but returned %s",
+                      smbcli_errstr(cli->tree)));
        }
 
        cli->session->vuid -= 1234;
 
-       if (NT_STATUS_IS_ERR(smbcli_close(cli->tree, fnum))) {
-               printf("close failed (%s)\n", smbcli_errstr(cli->tree));
-               correct = False;
-       }
+       torture_assert_ntstatus_ok(tctx, smbcli_close(cli->tree, fnum),
+               talloc_asprintf(tctx, "close failed (%s)", smbcli_errstr(cli->tree)));
 
        smbcli_unlink(cli->tree, fname);
 
-       if (!torture_close_connection(cli)) {
-               correct = False;
-       }
-
-       printf("vuid test finished\n");
-
-       return correct;
+       return true;
 }
 
 /*
   Test open mode returns on read-only files.
  */
- static BOOL run_opentest(struct torture_context *torture)
+ static bool run_opentest(struct torture_context *tctx, struct smbcli_state *cli1, 
+                                                 struct smbcli_state *cli2)
 {
-       static struct smbcli_state *cli1;
-       static struct smbcli_state *cli2;
        const char *fname = "\\readonly.file";
        char *control_char_fname;
        int fnum1, fnum2;
        uint8_t buf[20];
        size_t fsize;
-       BOOL correct = True;
+       bool correct = true;
        char *tmp_path;
        int failures = 0;
        int i;
 
-       printf("starting open test\n");
-       
-       if (!torture_open_connection(&cli1)) {
-               return False;
-       }
-       
        asprintf(&control_char_fname, "\\readonly.afile");
        for (i = 1; i <= 0x1f; i++) {
                control_char_fname[10] = i;
@@ -889,7 +765,7 @@ static BOOL run_vuidtest(struct torture_context *torture)
                
                if (!check_error(__location__, cli1, ERRDOS, ERRinvalidname, 
                                NT_STATUS_OBJECT_NAME_INVALID)) {
-                       printf("Error code should be NT_STATUS_OBJECT_NAME_INVALID, was %s for file with %d char\n",
+                       torture_comment(tctx, "Error code should be NT_STATUS_OBJECT_NAME_INVALID, was %s for file with %d char\n",
                                        smbcli_errstr(cli1->tree), i);
                        failures++;
                }
@@ -903,33 +779,33 @@ static BOOL run_vuidtest(struct torture_context *torture)
        free(control_char_fname);
 
        if (!failures)
-               printf("Create file with control char names passed.\n");
+               torture_comment(tctx, "Create file with control char names passed.\n");
 
        smbcli_setatr(cli1->tree, fname, 0, 0);
        smbcli_unlink(cli1->tree, fname);
        
        fnum1 = smbcli_open(cli1->tree, fname, O_RDWR|O_CREAT|O_EXCL, DENY_NONE);
        if (fnum1 == -1) {
-               printf("open of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "open of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("close2 failed (%s)\n", smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "close2 failed (%s)\n", smbcli_errstr(cli1->tree));
+               return false;
        }
        
        if (NT_STATUS_IS_ERR(smbcli_setatr(cli1->tree, fname, FILE_ATTRIBUTE_READONLY, 0))) {
-               printf("smbcli_setatr failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "smbcli_setatr failed (%s)\n", smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test1);
-               return False;
+               return false;
        }
        
        fnum1 = smbcli_open(cli1->tree, fname, O_RDONLY, DENY_WRITE);
        if (fnum1 == -1) {
-               printf("open of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "open of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test1);
-               return False;
+               return false;
        }
        
        /* This will fail - but the error should be ERRnoaccess, not ERRbadshare. */
@@ -937,10 +813,10 @@ static BOOL run_vuidtest(struct torture_context *torture)
        
         if (check_error(__location__, cli1, ERRDOS, ERRnoaccess, 
                        NT_STATUS_ACCESS_DENIED)) {
-               printf("correct error code ERRDOS/ERRnoaccess returned\n");
+               torture_comment(tctx, "correct error code ERRDOS/ERRnoaccess returned\n");
        }
        
-       printf("finished open test 1\n");
+       torture_comment(tctx, "finished open test 1\n");
 error_test1:
        smbcli_close(cli1->tree, fnum1);
        
@@ -950,8 +826,8 @@ error_test1:
        
        fnum1 = smbcli_open(cli1->tree, fname, O_RDONLY, DENY_WRITE);
        if (fnum1 == -1) {
-               printf("open of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "open of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
        
        /* This will fail - but the error should be ERRshare. */
@@ -959,24 +835,24 @@ error_test1:
        
        if (check_error(__location__, cli1, ERRDOS, ERRbadshare, 
                        NT_STATUS_SHARING_VIOLATION)) {
-               printf("correct error code ERRDOS/ERRbadshare returned\n");
+               torture_comment(tctx, "correct error code ERRDOS/ERRbadshare returned\n");
        }
        
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("close2 failed (%s)\n", smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "close2 failed (%s)\n", smbcli_errstr(cli1->tree));
+               return false;
        }
        
        smbcli_unlink(cli1->tree, fname);
        
-       printf("finished open test 2\n");
+       torture_comment(tctx, "finished open test 2\n");
        
        /* Test truncate open disposition on file opened for read. */
        
        fnum1 = smbcli_open(cli1->tree, fname, O_RDWR|O_CREAT|O_EXCL, DENY_NONE);
        if (fnum1 == -1) {
-               printf("(3) open (1) of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "(3) open (1) of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
        
        /* write 20 bytes. */
@@ -984,245 +860,241 @@ error_test1:
        memset(buf, '\0', 20);
 
        if (smbcli_write(cli1->tree, fnum1, 0, buf, 0, 20) != 20) {
-               printf("write failed (%s)\n", smbcli_errstr(cli1->tree));
-               correct = False;
+               torture_comment(tctx, "write failed (%s)\n", smbcli_errstr(cli1->tree));
+               correct = false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("(3) close1 failed (%s)\n", smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "(3) close1 failed (%s)\n", smbcli_errstr(cli1->tree));
+               return false;
        }
        
        /* Ensure size == 20. */
        if (NT_STATUS_IS_ERR(smbcli_getatr(cli1->tree, fname, NULL, &fsize, NULL))) {
-               printf("(3) getatr failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "(3) getatr failed (%s)\n", smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test3);
-               return False;
+               return false;
        }
        
        if (fsize != 20) {
-               printf("(3) file size != 20\n");
+               torture_comment(tctx, "(3) file size != 20\n");
                CHECK_MAX_FAILURES(error_test3);
-               return False;
+               return false;
        }
 
        /* Now test if we can truncate a file opened for readonly. */
        
        fnum1 = smbcli_open(cli1->tree, fname, O_RDONLY|O_TRUNC, DENY_NONE);
        if (fnum1 == -1) {
-               printf("(3) open (2) of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "(3) open (2) of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test3);
-               return False;
+               return false;
        }
        
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("close2 failed (%s)\n", smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "close2 failed (%s)\n", smbcli_errstr(cli1->tree));
+               return false;
        }
 
        /* Ensure size == 0. */
        if (NT_STATUS_IS_ERR(smbcli_getatr(cli1->tree, fname, NULL, &fsize, NULL))) {
-               printf("(3) getatr failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "(3) getatr failed (%s)\n", smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test3);
-               return False;
+               return false;
        }
 
        if (fsize != 0) {
-               printf("(3) file size != 0\n");
+               torture_comment(tctx, "(3) file size != 0\n");
                CHECK_MAX_FAILURES(error_test3);
-               return False;
+               return false;
        }
-       printf("finished open test 3\n");
+       torture_comment(tctx, "finished open test 3\n");
 error_test3:   
        smbcli_unlink(cli1->tree, fname);
 
 
-       printf("testing ctemp\n");
+       torture_comment(tctx, "testing ctemp\n");
        fnum1 = smbcli_ctemp(cli1->tree, "\\", &tmp_path);
        if (fnum1 == -1) {
-               printf("ctemp failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "ctemp failed (%s)\n", smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test4);
-               return False;
+               return false;
        }
-       printf("ctemp gave path %s\n", tmp_path);
+       torture_comment(tctx, "ctemp gave path %s\n", tmp_path);
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("close of temp failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "close of temp failed (%s)\n", smbcli_errstr(cli1->tree));
        }
        if (NT_STATUS_IS_ERR(smbcli_unlink(cli1->tree, tmp_path))) {
-               printf("unlink of temp failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "unlink of temp failed (%s)\n", smbcli_errstr(cli1->tree));
        }
 error_test4:   
        /* Test the non-io opens... */
 
-       if (!torture_open_connection(&cli2)) {
-               return False;
-       }
-       
        smbcli_setatr(cli2->tree, fname, 0, 0);
        smbcli_unlink(cli2->tree, fname);
        
-       printf("TEST #1 testing 2 non-io opens (no delete)\n");
+       torture_comment(tctx, "TEST #1 testing 2 non-io opens (no delete)\n");
        
        fnum1 = smbcli_nt_create_full(cli1->tree, fname, 0, SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OVERWRITE_IF, 0, 0);
 
        if (fnum1 == -1) {
-               printf("test 1 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "test 1 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test10);
-               return False;
+               return false;
        }
 
        fnum2 = smbcli_nt_create_full(cli2->tree, fname, 0, SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OPEN_IF, 0, 0);
        if (fnum2 == -1) {
-               printf("test 1 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               torture_comment(tctx, "test 1 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
                CHECK_MAX_FAILURES(error_test10);
-               return False;
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("test 1 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "test 1 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
        if (NT_STATUS_IS_ERR(smbcli_close(cli2->tree, fnum2))) {
-               printf("test 1 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
-               return False;
+               torture_comment(tctx, "test 1 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               return false;
        }
 
-       printf("non-io open test #1 passed.\n");
+       torture_comment(tctx, "non-io open test #1 passed.\n");
 error_test10:
        smbcli_unlink(cli1->tree, fname);
 
-       printf("TEST #2 testing 2 non-io opens (first with delete)\n");
+       torture_comment(tctx, "TEST #2 testing 2 non-io opens (first with delete)\n");
        
        fnum1 = smbcli_nt_create_full(cli1->tree, fname, 0, SEC_STD_DELETE|SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OVERWRITE_IF, 0, 0);
 
        if (fnum1 == -1) {
-               printf("test 2 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "test 2 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test20);
-               return False;
+               return false;
        }
 
        fnum2 = smbcli_nt_create_full(cli2->tree, fname, 0, SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OPEN_IF, 0, 0);
 
        if (fnum2 == -1) {
-               printf("test 2 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               torture_comment(tctx, "test 2 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
                CHECK_MAX_FAILURES(error_test20);
-               return False;
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("test 1 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "test 1 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
        if (NT_STATUS_IS_ERR(smbcli_close(cli2->tree, fnum2))) {
-               printf("test 1 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "test 1 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
 
-       printf("non-io open test #2 passed.\n");
+       torture_comment(tctx, "non-io open test #2 passed.\n");
 error_test20:
        smbcli_unlink(cli1->tree, fname);
 
-       printf("TEST #3 testing 2 non-io opens (second with delete)\n");
+       torture_comment(tctx, "TEST #3 testing 2 non-io opens (second with delete)\n");
        
        fnum1 = smbcli_nt_create_full(cli1->tree, fname, 0, SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OVERWRITE_IF, 0, 0);
 
        if (fnum1 == -1) {
-               printf("test 3 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "test 3 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test30);
-               return False;
+               return false;
        }
 
        fnum2 = smbcli_nt_create_full(cli2->tree, fname, 0, SEC_STD_DELETE|SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OPEN_IF, 0, 0);
 
        if (fnum2 == -1) {
-               printf("test 3 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               torture_comment(tctx, "test 3 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
                CHECK_MAX_FAILURES(error_test30);
-               return False;
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("test 3 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "test 3 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
        if (NT_STATUS_IS_ERR(smbcli_close(cli2->tree, fnum2))) {
-               printf("test 3 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
-               return False;
+               torture_comment(tctx, "test 3 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               return false;
        }
 
-       printf("non-io open test #3 passed.\n");
+       torture_comment(tctx, "non-io open test #3 passed.\n");
 error_test30:
        smbcli_unlink(cli1->tree, fname);
 
-       printf("TEST #4 testing 2 non-io opens (both with delete)\n");
+       torture_comment(tctx, "TEST #4 testing 2 non-io opens (both with delete)\n");
        
        fnum1 = smbcli_nt_create_full(cli1->tree, fname, 0, SEC_STD_DELETE|SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OVERWRITE_IF, 0, 0);
 
        if (fnum1 == -1) {
-               printf("test 4 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "test 4 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test40);
-               return False;
+               return false;
        }
 
        fnum2 = smbcli_nt_create_full(cli2->tree, fname, 0, SEC_STD_DELETE|SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OPEN_IF, 0, 0);
 
        if (fnum2 != -1) {
-               printf("test 4 open 2 of %s SUCCEEDED - should have failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               torture_comment(tctx, "test 4 open 2 of %s SUCCEEDED - should have failed (%s)\n", fname, smbcli_errstr(cli2->tree));
                CHECK_MAX_FAILURES(error_test40);
-               return False;
+               return false;
        }
 
-       printf("test 4 open 2 of %s gave %s (correct error should be %s)\n", fname, smbcli_errstr(cli2->tree), "sharing violation");
+       torture_comment(tctx, "test 4 open 2 of %s gave %s (correct error should be %s)\n", fname, smbcli_errstr(cli2->tree), "sharing violation");
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("test 4 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "test 4 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
 
-       printf("non-io open test #4 passed.\n");
+       torture_comment(tctx, "non-io open test #4 passed.\n");
 error_test40:
        smbcli_unlink(cli1->tree, fname);
 
-       printf("TEST #5 testing 2 non-io opens (both with delete - both with file share delete)\n");
+       torture_comment(tctx, "TEST #5 testing 2 non-io opens (both with delete - both with file share delete)\n");
        
        fnum1 = smbcli_nt_create_full(cli1->tree, fname, 0, SEC_STD_DELETE|SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_DELETE, NTCREATEX_DISP_OVERWRITE_IF, 0, 0);
 
        if (fnum1 == -1) {
-               printf("test 5 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "test 5 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test50);
-               return False;
+               return false;
        }
 
        fnum2 = smbcli_nt_create_full(cli2->tree, fname, 0, SEC_STD_DELETE|SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_DELETE, NTCREATEX_DISP_OPEN_IF, 0, 0);
 
        if (fnum2 == -1) {
-               printf("test 5 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               torture_comment(tctx, "test 5 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
                CHECK_MAX_FAILURES(error_test50);
-               return False;
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("test 5 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "test 5 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli2->tree, fnum2))) {
-               printf("test 5 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
-               return False;
+               torture_comment(tctx, "test 5 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               return false;
        }
 
-       printf("non-io open test #5 passed.\n");
+       torture_comment(tctx, "non-io open test #5 passed.\n");
 error_test50:
-       printf("TEST #6 testing 1 non-io open, one io open\n");
+       torture_comment(tctx, "TEST #6 testing 1 non-io open, one io open\n");
        
        smbcli_unlink(cli1->tree, fname);
 
@@ -1230,33 +1102,33 @@ error_test50:
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OVERWRITE_IF, 0, 0);
 
        if (fnum1 == -1) {
-               printf("test 6 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "test 6 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test60);
-               return False;
+               return false;
        }
 
        fnum2 = smbcli_nt_create_full(cli2->tree, fname, 0, SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_READ, NTCREATEX_DISP_OPEN_IF, 0, 0);
 
        if (fnum2 == -1) {
-               printf("test 6 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               torture_comment(tctx, "test 6 open 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
                CHECK_MAX_FAILURES(error_test60);
-               return False;
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("test 6 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "test 6 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli2->tree, fnum2))) {
-               printf("test 6 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
-               return False;
+               torture_comment(tctx, "test 6 close 2 of %s failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               return false;
        }
 
-       printf("non-io open test #6 passed.\n");
+       torture_comment(tctx, "non-io open test #6 passed.\n");
 error_test60:
-       printf("TEST #7 testing 1 non-io open, one io open with delete\n");
+       torture_comment(tctx, "TEST #7 testing 1 non-io open, one io open with delete\n");
 
        smbcli_unlink(cli1->tree, fname);
 
@@ -1264,39 +1136,39 @@ error_test60:
                                   NTCREATEX_SHARE_ACCESS_NONE, NTCREATEX_DISP_OVERWRITE_IF, 0, 0);
 
        if (fnum1 == -1) {
-               printf("test 7 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "test 7 open 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test70);
-               return False;
+               return false;
        }
 
        fnum2 = smbcli_nt_create_full(cli2->tree, fname, 0, SEC_STD_DELETE|SEC_FILE_READ_ATTRIBUTE, FILE_ATTRIBUTE_NORMAL,
                                   NTCREATEX_SHARE_ACCESS_READ|NTCREATEX_SHARE_ACCESS_DELETE, NTCREATEX_DISP_OPEN_IF, 0, 0);
 
        if (fnum2 != -1) {
-               printf("test 7 open 2 of %s SUCCEEDED - should have failed (%s)\n", fname, smbcli_errstr(cli2->tree));
+               torture_comment(tctx, "test 7 open 2 of %s SUCCEEDED - should have failed (%s)\n", fname, smbcli_errstr(cli2->tree));
                CHECK_MAX_FAILURES(error_test70);
-               return False;
+               return false;
        }
 
-       printf("test 7 open 2 of %s gave %s (correct error should be %s)\n", fname, smbcli_errstr(cli2->tree), "sharing violation");
+       torture_comment(tctx, "test 7 open 2 of %s gave %s (correct error should be %s)\n", fname, smbcli_errstr(cli2->tree), "sharing violation");
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("test 7 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "test 7 close 1 of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
 
-       printf("non-io open test #7 passed.\n");
+       torture_comment(tctx, "non-io open test #7 passed.\n");
 
 error_test70:
 
-       printf("TEST #8 testing one normal open, followed by lock, followed by open with truncate\n");
+       torture_comment(tctx, "TEST #8 testing one normal open, followed by lock, followed by open with truncate\n");
 
        smbcli_unlink(cli1->tree, fname);
 
        fnum1 = smbcli_open(cli1->tree, fname, O_RDWR|O_CREAT, DENY_NONE);
        if (fnum1 == -1) {
-               printf("(8) open (1) of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "(8) open (1) of %s failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
        
        /* write 20 bytes. */
@@ -1304,72 +1176,65 @@ error_test70:
        memset(buf, '\0', 20);
 
        if (smbcli_write(cli1->tree, fnum1, 0, buf, 0, 20) != 20) {
-               printf("(8) write failed (%s)\n", smbcli_errstr(cli1->tree));
-               correct = False;
+               torture_comment(tctx, "(8) write failed (%s)\n", smbcli_errstr(cli1->tree));
+               correct = false;
        }
 
        /* Ensure size == 20. */
        if (NT_STATUS_IS_ERR(smbcli_getatr(cli1->tree, fname, NULL, &fsize, NULL))) {
-               printf("(8) getatr (1) failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "(8) getatr (1) failed (%s)\n", smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test80);
-               return False;
+               return false;
        }
        
        if (fsize != 20) {
-               printf("(8) file size != 20\n");
+               torture_comment(tctx, "(8) file size != 20\n");
                CHECK_MAX_FAILURES(error_test80);
-               return False;
+               return false;
        }
 
        /* Get an exclusive lock on the open file. */
        if (NT_STATUS_IS_ERR(smbcli_lock(cli1->tree, fnum1, 0, 4, 0, WRITE_LOCK))) {
-               printf("(8) lock1 failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "(8) lock1 failed (%s)\n", smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test80);
-               return False;
+               return false;
        }
 
        fnum2 = smbcli_open(cli1->tree, fname, O_RDWR|O_TRUNC, DENY_NONE);
        if (fnum1 == -1) {
-               printf("(8) open (2) of %s with truncate failed (%s)\n", fname, smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "(8) open (2) of %s with truncate failed (%s)\n", fname, smbcli_errstr(cli1->tree));
+               return false;
        }
 
        /* Ensure size == 0. */
        if (NT_STATUS_IS_ERR(smbcli_getatr(cli1->tree, fname, NULL, &fsize, NULL))) {
-               printf("(8) getatr (2) failed (%s)\n", smbcli_errstr(cli1->tree));
+               torture_comment(tctx, "(8) getatr (2) failed (%s)\n", smbcli_errstr(cli1->tree));
                CHECK_MAX_FAILURES(error_test80);
-               return False;
+               return false;
        }
        
        if (fsize != 0) {
-               printf("(8) file size != 0\n");
+               torture_comment(tctx, "(8) file size != 0\n");
                CHECK_MAX_FAILURES(error_test80);
-               return False;
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum1))) {
-               printf("(8) close1 failed (%s)\n", smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "(8) close1 failed (%s)\n", smbcli_errstr(cli1->tree));
+               return false;
        }
        
        if (NT_STATUS_IS_ERR(smbcli_close(cli1->tree, fnum2))) {
-               printf("(8) close1 failed (%s)\n", smbcli_errstr(cli1->tree));
-               return False;
+               torture_comment(tctx, "(8) close1 failed (%s)\n", smbcli_errstr(cli1->tree));
+               return false;
        }
        
 error_test80:
 
-       printf("open test #8 passed.\n");
+       torture_comment(tctx, "open test #8 passed.\n");
 
        smbcli_unlink(cli1->tree, fname);
 
-       if (!torture_close_connection(cli1)) {
-               correct = False;
-       }
-       if (!torture_close_connection(cli2)) {
-               correct = False;
-       }
-       
        return correct;
 }
 
@@ -1397,19 +1262,12 @@ error_test80:
 /**
   Test ntcreate calls made by xcopy
  */
-static BOOL run_xcopy(struct torture_context *torture)
+static bool run_xcopy(struct torture_context *tctx,
+                                         struct smbcli_state *cli1)
 {
-       struct smbcli_state *cli1;
        const char *fname = "\\test.txt";
-       BOOL correct = True;
        int fnum1, fnum2;
 
-       printf("starting xcopy test\n");
-       
-       if (!torture_open_connection(&cli1)) {
-               return False;
-       }
-       
        fnum1 = smbcli_nt_create_full(cli1->tree, fname, 0,
                                      FIRST_DESIRED_ACCESS, 
                                      FILE_ATTRIBUTE_ARCHIVE,
@@ -1417,66 +1275,47 @@ static BOOL run_xcopy(struct torture_context *torture)
                                      NTCREATEX_DISP_OVERWRITE_IF, 
                                      0x4044, 0);
 
-       if (fnum1 == -1) {
-               printf("First open failed - %s\n", smbcli_errstr(cli1->tree));
-               return False;
-       }
+       torture_assert(tctx, fnum1 != -1, talloc_asprintf(tctx, 
+                               "First open failed - %s", smbcli_errstr(cli1->tree)));
 
        fnum2 = smbcli_nt_create_full(cli1->tree, fname, 0,
                                   SECOND_DESIRED_ACCESS, 0,
                                   NTCREATEX_SHARE_ACCESS_READ|NTCREATEX_SHARE_ACCESS_WRITE|NTCREATEX_SHARE_ACCESS_DELETE, NTCREATEX_DISP_OPEN, 
                                   0x200000, 0);
-       if (fnum2 == -1) {
-               printf("second open failed - %s\n", smbcli_errstr(cli1->tree));
-               return False;
-       }
-       
-       if (!torture_close_connection(cli1)) {
-               correct = False;
-       }
+       torture_assert(tctx, fnum2 != -1, talloc_asprintf(tctx, 
+                               "second open failed - %s", smbcli_errstr(cli1->tree)));
        
-       return correct;
+       return true;
 }
 
-static BOOL run_iometer(struct torture_context *torture)
+static bool run_iometer(struct torture_context *tctx,
+                                               struct smbcli_state *cli)
 {
-       struct smbcli_state *cli;
        const char *fname = "\\iobw.tst";
        int fnum;
        size_t filesize;
        NTSTATUS status;
        char buf[2048];
        int ops;
-       BOOL result = False;
-
-       printf("Starting iometer test\n");
 
        memset(buf, 0, sizeof(buf));
 
-       if (!torture_open_connection(&cli)) {
-               return False;
-       }
-
        status = smbcli_getatr(cli->tree, fname, NULL, &filesize, NULL);
-       if (!NT_STATUS_IS_OK(status)) {
-               printf("smbcli_getatr failed: %s\n", nt_errstr(status));
-               goto done;
-       }
+       torture_assert_ntstatus_ok(tctx, status, 
+               talloc_asprintf(tctx, "smbcli_getatr failed: %s", nt_errstr(status)));
 
-       printf("size: %d\n", filesize);
+       torture_comment(tctx, "size: %d\n", (int)filesize);
 
        filesize -= (sizeof(buf) - 1);
 
        fnum = smbcli_nt_create_full(cli->tree, fname, 0x16,
                                     0x2019f, 0, 0x3, 3, 0x42, 0x3);
-       if (fnum == -1) {
-               printf("open failed: %s\n", smbcli_errstr(cli->tree));
-               goto done;
-       }
+       torture_assert(tctx, fnum != -1, talloc_asprintf(tctx, "open failed: %s", 
+                                  smbcli_errstr(cli->tree)));
 
        ops = 0;
 
-       while (True) {
+       while (true) {
                int i, num_reads, num_writes;
 
                num_reads = random() % 10;
@@ -1485,61 +1324,40 @@ static BOOL run_iometer(struct torture_context *torture)
                for (i=0; i<num_reads; i++) {
                        ssize_t res;
                        if (ops++ > torture_numops) {
-                               result = True;
-                               goto done;
+                               return true;
                        }
                        res = smbcli_read(cli->tree, fnum, buf,
                                          random() % filesize, sizeof(buf));
-                       if (res != sizeof(buf)) {
-                               printf("read failed: %s\n",
-                                      smbcli_errstr(cli->tree));
-                               goto done;
-                       }
+                       torture_assert(tctx, res == sizeof(buf), 
+                                                  talloc_asprintf(tctx, "read failed: %s",
+                                                                                  smbcli_errstr(cli->tree)));
                }
                for (i=0; i<num_writes; i++) {
                        ssize_t res;
                        if (ops++ > torture_numops) {
-                               result = True;
-                               goto done;
+                               return true;
                        }
                        res = smbcli_write(cli->tree, fnum, 0, buf,
                                          random() % filesize, sizeof(buf));
-                       if (res != sizeof(buf)) {
-                               printf("read failed: %s\n",
-                                      smbcli_errstr(cli->tree));
-                               goto done;
-                       }
+                       torture_assert(tctx, res == sizeof(buf),
+                                                  talloc_asprintf(tctx, "read failed: %s",
+                                      smbcli_errstr(cli->tree)));
                }
        }
 
-       result = True;
- done:
-
-       if (!torture_close_connection(cli)) {
-               printf("close_connection failed: %s\n",
-                      smbcli_errstr(cli->tree));
-               return False;
-       }
-
-       return result;
+       return true;
 }
 
 /**
   tries variants of chkpath
  */
-static BOOL torture_chkpath_test(struct torture_context *torture)
+static bool torture_chkpath_test(struct torture_context *tctx, 
+                                                                struct smbcli_state *cli)
 {
-       struct smbcli_state *cli;
        int fnum;
-       BOOL ret;
-
-       if (!torture_open_connection(&cli)) {
-               return False;
-       }
+       bool ret;
 
-       printf("starting chkpath test\n");
-
-       printf("Testing valid and invalid paths\n");
+       torture_comment(tctx, "Testing valid and invalid paths\n");
 
        /* cleanup from an old run */
        smbcli_rmdir(cli->tree, "\\chkpath.dir\\dir2");
@@ -1547,64 +1365,60 @@ static BOOL torture_chkpath_test(struct torture_context *torture)
        smbcli_rmdir(cli->tree, "\\chkpath.dir");
 
        if (NT_STATUS_IS_ERR(smbcli_mkdir(cli->tree, "\\chkpath.dir"))) {
-               printf("mkdir1 failed : %s\n", smbcli_errstr(cli->tree));
-               return False;
+               torture_comment(tctx, "mkdir1 failed : %s\n", smbcli_errstr(cli->tree));
+               return false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_mkdir(cli->tree, "\\chkpath.dir\\dir2"))) {
-               printf("mkdir2 failed : %s\n", smbcli_errstr(cli->tree));
-               return False;
+               torture_comment(tctx, "mkdir2 failed : %s\n", smbcli_errstr(cli->tree));
+               return false;
        }
 
        fnum = smbcli_open(cli->tree, "\\chkpath.dir\\foo.txt", O_RDWR|O_CREAT|O_EXCL, DENY_NONE);
        if (fnum == -1) {
-               printf("open1 failed (%s)\n", smbcli_errstr(cli->tree));
-               return False;
+               torture_comment(tctx, "open1 failed (%s)\n", smbcli_errstr(cli->tree));
+               return false;
        }
        smbcli_close(cli->tree, fnum);
 
        if (NT_STATUS_IS_ERR(smbcli_chkpath(cli->tree, "\\chkpath.dir"))) {
-               printf("chkpath1 failed: %s\n", smbcli_errstr(cli->tree));
-               ret = False;
+               torture_comment(tctx, "chkpath1 failed: %s\n", smbcli_errstr(cli->tree));
+               ret = false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_chkpath(cli->tree, "\\chkpath.dir\\dir2"))) {
-               printf("chkpath2 failed: %s\n", smbcli_errstr(cli->tree));
-               ret = False;
+               torture_comment(tctx, "chkpath2 failed: %s\n", smbcli_errstr(cli->tree));
+               ret = false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_chkpath(cli->tree, "\\chkpath.dir\\foo.txt"))) {
                ret = check_error(__location__, cli, ERRDOS, ERRbadpath, 
                                  NT_STATUS_NOT_A_DIRECTORY);
        } else {
-               printf("* chkpath on a file should fail\n");
-               ret = False;
+               torture_comment(tctx, "* chkpath on a file should fail\n");
+               ret = false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_chkpath(cli->tree, "\\chkpath.dir\\bar.txt"))) {
                ret = check_error(__location__, cli, ERRDOS, ERRbadpath, 
                                  NT_STATUS_OBJECT_NAME_NOT_FOUND);
        } else {
-               printf("* chkpath on a non existent file should fail\n");
-               ret = False;
+               torture_comment(tctx, "* chkpath on a non existent file should fail\n");
+               ret = false;
        }
 
        if (NT_STATUS_IS_ERR(smbcli_chkpath(cli->tree, "\\chkpath.dir\\dirxx\\bar.txt"))) {
                ret = check_error(__location__, cli, ERRDOS, ERRbadpath, 
                                  NT_STATUS_OBJECT_PATH_NOT_FOUND);
        } else {
-               printf("* chkpath on a non existent component should fail\n");
-               ret = False;
+               torture_comment(tctx, "* chkpath on a non existent component should fail\n");
+               ret = false;
        }
 
        smbcli_rmdir(cli->tree, "\\chkpath.dir\\dir2");
        smbcli_unlink(cli->tree, "\\chkpath.dir\\*");
        smbcli_rmdir(cli->tree, "\\chkpath.dir");
 
-       if (!torture_close_connection(cli)) {
-               return False;
-       }
-
        return ret;
 }
 
@@ -1612,54 +1426,54 @@ static BOOL torture_chkpath_test(struct torture_context *torture)
  * This is a test to excercise some weird Samba3 error paths.
  */
 
-static BOOL torture_samba3_errorpaths(struct torture_context *torture)
+static bool torture_samba3_errorpaths(struct torture_context *tctx)
 {
-       BOOL nt_status_support;
+       bool nt_status_support;
        struct smbcli_state *cli_nt = NULL, *cli_dos = NULL;
-       BOOL result = False;
+       bool result = false;
        int fnum;
        const char *os2_fname = ".+,;=[].";
-       const char *dirname = "samba3_errordir";
+       const char *dname = "samba3_errordir";
        union smb_open io;
-       TALLOC_CTX *mem_ctx = talloc_init(NULL);
+       TALLOC_CTX *mem_ctx = talloc_init("samba3_errorpaths");
        NTSTATUS status;
 
        if (mem_ctx == NULL) {
-               printf("talloc_init failed\n");
-               return False;
+               torture_comment(tctx, "talloc_init failed\n");
+               return false;
        }
 
-       nt_status_support = lp_nt_status_support();
+       nt_status_support = lp_nt_status_support(tctx->lp_ctx);
 
-       if (!lp_set_cmdline("nt status support", "yes")) {
-               printf("Could not set 'nt status support = yes'\n");
+       if (!lp_set_cmdline(tctx->lp_ctx, "nt status support", "yes")) {
+               torture_comment(tctx, "Could not set 'nt status support = yes'\n");
                goto fail;
        }
 
-       if (!torture_open_connection(&cli_nt)) {
+       if (!torture_open_connection(&cli_nt, tctx, 0)) {
                goto fail;
        }
 
-       if (!lp_set_cmdline("nt status support", "no")) {
-               printf("Could not set 'nt status support = yes'\n");
+       if (!lp_set_cmdline(tctx->lp_ctx, "nt status support", "no")) {
+               torture_comment(tctx, "Could not set 'nt status support = yes'\n");
                goto fail;
        }
 
-       if (!torture_open_connection(&cli_dos)) {
+       if (!torture_open_connection(&cli_dos, tctx, 1)) {
                goto fail;
        }
 
-       if (!lp_set_cmdline("nt status support",
+       if (!lp_set_cmdline(tctx->lp_ctx, "nt status support",
                            nt_status_support ? "yes":"no")) {
-               printf("Could not reset 'nt status support = yes'");
+               torture_comment(tctx, "Could not reset 'nt status support = yes'");
                goto fail;
        }
 
        smbcli_unlink(cli_nt->tree, os2_fname);
-       smbcli_rmdir(cli_nt->tree, dirname);
+       smbcli_rmdir(cli_nt->tree, dname);
 
-       if (!NT_STATUS_IS_OK(smbcli_mkdir(cli_nt->tree, dirname))) {
-               printf("smbcli_mkdir(%s) failed: %s\n", dirname,
+       if (!NT_STATUS_IS_OK(smbcli_mkdir(cli_nt->tree, dname))) {
+               torture_comment(tctx, "smbcli_mkdir(%s) failed: %s\n", dname,
                       smbcli_errstr(cli_nt->tree));
                goto fail;
        }
@@ -1675,39 +1489,176 @@ static BOOL torture_samba3_errorpaths(struct torture_context *torture)
        io.ntcreatex.in.create_options = 0;
        io.ntcreatex.in.impersonation = NTCREATEX_IMPERSONATION_ANONYMOUS;
        io.ntcreatex.in.security_flags = 0;
-       io.ntcreatex.in.fname = dirname;
+       io.ntcreatex.in.fname = dname;
 
        status = smb_raw_open(cli_nt->tree, mem_ctx, &io);
        if (!NT_STATUS_EQUAL(status, NT_STATUS_OBJECT_NAME_COLLISION)) {
-               printf("(%s) incorrect status %s should be %s\n",
+               torture_comment(tctx, "(%s) incorrect status %s should be %s\n",
                       __location__, nt_errstr(status),
                       nt_errstr(NT_STATUS_OBJECT_NAME_COLLISION));
                goto fail;
        }
        status = smb_raw_open(cli_dos->tree, mem_ctx, &io);
        if (!NT_STATUS_EQUAL(status, NT_STATUS_DOS(ERRDOS, ERRfilexists))) {
-               printf("(%s) incorrect status %s should be %s\n",
+               torture_comment(tctx, "(%s) incorrect status %s should be %s\n",
                       __location__, nt_errstr(status),
                       nt_errstr(NT_STATUS_DOS(ERRDOS, ERRfilexists)));
                goto fail;
        }
 
-       status = smbcli_mkdir(cli_nt->tree, dirname);
+       status = smbcli_mkdir(cli_nt->tree, dname);
        if (!NT_STATUS_EQUAL(status, NT_STATUS_OBJECT_NAME_COLLISION)) {
-               printf("(%s) incorrect status %s should be %s\n",
+               torture_comment(tctx, "(%s) incorrect status %s should be %s\n",
                       __location__, nt_errstr(status),
                       nt_errstr(NT_STATUS_OBJECT_NAME_COLLISION));
                goto fail;
        }
-       status = smbcli_mkdir(cli_dos->tree, dirname);
+       status = smbcli_mkdir(cli_dos->tree, dname);
        if (!NT_STATUS_EQUAL(status, NT_STATUS_DOS(ERRDOS, ERRnoaccess))) {
-               printf("(%s) incorrect status %s should be %s\n",
+               torture_comment(tctx, "(%s) incorrect status %s should be %s\n",
                       __location__, nt_errstr(status),
                       nt_errstr(NT_STATUS_DOS(ERRDOS, ERRnoaccess)));
                goto fail;
        }
 
-       if (!lp_parm_bool(-1, "target", "samba3", False)) {
+       {
+               union smb_mkdir md;
+               md.t2mkdir.level = RAW_MKDIR_T2MKDIR;
+               md.t2mkdir.in.path = dname;
+               md.t2mkdir.in.num_eas = 0;
+               md.t2mkdir.in.eas = NULL;
+
+               status = smb_raw_mkdir(cli_nt->tree, &md);
+               if (!NT_STATUS_EQUAL(status,
+                                    NT_STATUS_OBJECT_NAME_COLLISION)) {
+                       torture_comment(
+                               tctx, "(%s) incorrect status %s should be "
+                               "NT_STATUS_OBJECT_NAME_COLLISION\n",
+                               __location__, nt_errstr(status));
+                       goto fail;
+               }
+               status = smb_raw_mkdir(cli_dos->tree, &md);
+               if (!NT_STATUS_EQUAL(status,
+                                    NT_STATUS_DOS(ERRDOS, ERRrename))) {
+                       torture_comment(tctx, "(%s) incorrect status %s "
+                                       "should be ERRDOS:ERRrename\n",
+                                       __location__, nt_errstr(status));
+                       goto fail;
+               }
+       }
+
+       io.ntcreatex.in.create_options = NTCREATEX_OPTIONS_DIRECTORY;
+       status = smb_raw_open(cli_nt->tree, mem_ctx, &io);
+       if (!NT_STATUS_EQUAL(status, NT_STATUS_OBJECT_NAME_COLLISION)) {
+               torture_comment(tctx, "(%s) incorrect status %s should be %s\n",
+                      __location__, nt_errstr(status),
+                      nt_errstr(NT_STATUS_OBJECT_NAME_COLLISION));
+               goto fail;
+       }
+
+       status = smb_raw_open(cli_dos->tree, mem_ctx, &io);
+       if (!NT_STATUS_EQUAL(status, NT_STATUS_DOS(ERRDOS, ERRfilexists))) {
+               torture_comment(tctx, "(%s) incorrect status %s should be %s\n",
+                      __location__, nt_errstr(status),
+                      nt_errstr(NT_STATUS_DOS(ERRDOS, ERRfilexists)));
+               goto fail;
+       }
+
+       {
+               /* Test an invalid DOS deny mode */
+               const char *fname = "test.txt";
+
+               fnum = smbcli_open(cli_nt->tree, fname, O_RDWR | O_CREAT, 5);
+               if (fnum != -1) {
+                       torture_comment(tctx, "Open(%s) with invalid deny mode succeeded -- "
+                              "expected failure\n", fname);
+                       smbcli_close(cli_nt->tree, fnum);
+                       goto fail;
+               }
+               if (!NT_STATUS_EQUAL(smbcli_nt_error(cli_nt->tree),
+                                    NT_STATUS_DOS(ERRDOS,ERRbadaccess))) {
+                       torture_comment(tctx, "Expected DOS error ERRDOS/ERRbadaccess, "
+                              "got %s\n", smbcli_errstr(cli_nt->tree));
+                       goto fail;
+               }
+
+               fnum = smbcli_open(cli_dos->tree, fname, O_RDWR | O_CREAT, 5);
+               if (fnum != -1) {
+                       torture_comment(tctx, "Open(%s) with invalid deny mode succeeded -- "
+                              "expected failure\n", fname);
+                       smbcli_close(cli_nt->tree, fnum);
+                       goto fail;
+               }
+               if (!NT_STATUS_EQUAL(smbcli_nt_error(cli_nt->tree),
+                                    NT_STATUS_DOS(ERRDOS,ERRbadaccess))) {
+                       torture_comment(tctx, "Expected DOS error ERRDOS:ERRbadaccess, "
+                              "got %s\n", smbcli_errstr(cli_nt->tree));
+                       goto fail;
+               }
+       }
+
+       {
+               /*
+                * Samba 3.0.23 has a bug that an existing file can be opened
+                * as a directory using ntcreate&x. Test this.
+                */
+
+               const char *fname = "\\test_dir.txt";
+
+               fnum = smbcli_open(cli_nt->tree, fname, O_RDWR|O_CREAT,
+                                  DENY_NONE);
+               if (fnum == -1) {
+                       d_printf("(%s) smbcli_open failed: %s\n", __location__,
+                                smbcli_errstr(cli_nt->tree));
+               }
+               smbcli_close(cli_nt->tree, fnum);
+
+               io.generic.level = RAW_OPEN_NTCREATEX;
+               io.ntcreatex.in.root_fid = 0;
+               io.ntcreatex.in.access_mask = SEC_RIGHTS_FILE_ALL;
+               io.ntcreatex.in.alloc_size = 0;
+               io.ntcreatex.in.file_attr = FILE_ATTRIBUTE_DIRECTORY;
+               io.ntcreatex.in.share_access = NTCREATEX_SHARE_ACCESS_READ|
+                       NTCREATEX_SHARE_ACCESS_WRITE|
+                       NTCREATEX_SHARE_ACCESS_DELETE;
+               io.ntcreatex.in.open_disposition = NTCREATEX_DISP_OPEN;
+               io.ntcreatex.in.create_options = NTCREATEX_OPTIONS_DIRECTORY;
+               io.ntcreatex.in.impersonation =
+                       NTCREATEX_IMPERSONATION_ANONYMOUS;
+               io.ntcreatex.in.security_flags = 0;
+               io.ntcreatex.in.fname = fname;
+               io.ntcreatex.in.flags = 0;
+
+               status = smb_raw_open(cli_nt->tree, mem_ctx, &io);
+               if (!NT_STATUS_EQUAL(status, NT_STATUS_NOT_A_DIRECTORY)) {
+                       torture_comment(tctx, "ntcreate as dir gave %s, "
+                                       "expected NT_STATUS_NOT_A_DIRECTORY\n",
+                                       nt_errstr(status));
+                       result = false;
+               }
+
+               if (NT_STATUS_IS_OK(status)) {
+                       smbcli_close(cli_nt->tree, io.ntcreatex.out.file.fnum);
+               }
+
+               status = smb_raw_open(cli_dos->tree, mem_ctx, &io);
+               if (!NT_STATUS_EQUAL(status, NT_STATUS_DOS(ERRDOS,
+                                                          ERRbaddirectory))) {
+                       torture_comment(tctx, "ntcreate as dir gave %s, "
+                                       "expected NT_STATUS_NOT_A_DIRECTORY\n",
+                                       nt_errstr(status));
+                       result = false;
+               }
+
+               if (NT_STATUS_IS_OK(status)) {
+                       smbcli_close(cli_dos->tree,
+                                    io.ntcreatex.out.file.fnum);
+               }
+
+               smbcli_unlink(cli_nt->tree, fname);
+       }
+
+       if (!torture_setting_bool(tctx, "samba3", false)) {
                goto done;
        }
 
@@ -1715,7 +1666,7 @@ static BOOL torture_samba3_errorpaths(struct torture_context *torture)
                           O_RDWR | O_CREAT | O_TRUNC,
                           DENY_NONE);
        if (fnum != -1) {
-               printf("Open(%s) succeeded -- expected failure\n",
+               torture_comment(tctx, "Open(%s) succeeded -- expected failure\n",
                       os2_fname);
                smbcli_close(cli_dos->tree, fnum);
                goto fail;
@@ -1723,7 +1674,7 @@ static BOOL torture_samba3_errorpaths(struct torture_context *torture)
 
        if (!NT_STATUS_EQUAL(smbcli_nt_error(cli_dos->tree),
                             NT_STATUS_DOS(ERRDOS, ERRcannotopen))) {
-               printf("Expected DOS error ERRDOS/ERRcannotopen, got %s\n",
+               torture_comment(tctx, "Expected DOS error ERRDOS/ERRcannotopen, got %s\n",
                       smbcli_errstr(cli_dos->tree));
                goto fail;
        }
@@ -1732,7 +1683,7 @@ static BOOL torture_samba3_errorpaths(struct torture_context *torture)
                           O_RDWR | O_CREAT | O_TRUNC,
                           DENY_NONE);
        if (fnum != -1) {
-               printf("Open(%s) succeeded -- expected failure\n",
+               torture_comment(tctx, "Open(%s) succeeded -- expected failure\n",
                       os2_fname);
                smbcli_close(cli_nt->tree, fnum);
                goto fail;
@@ -1740,13 +1691,13 @@ static BOOL torture_samba3_errorpaths(struct torture_context *torture)
 
        if (!NT_STATUS_EQUAL(smbcli_nt_error(cli_nt->tree),
                             NT_STATUS_OBJECT_NAME_NOT_FOUND)) {
-               printf("Expected DOS error NT_STATUS_OBJECT_NAME_NOT_FOUND, "
+               torture_comment(tctx, "Expected error NT_STATUS_OBJECT_NAME_NOT_FOUND, "
                       "got %s\n", smbcli_errstr(cli_nt->tree));
                goto fail;
        }
 
  done:
-       result = True;
+       result = true;
 
  fail:
        if (cli_dos != NULL) {
@@ -1762,52 +1713,59 @@ static BOOL torture_samba3_errorpaths(struct torture_context *torture)
 
 NTSTATUS torture_base_init(void)
 {
-       register_torture_op("BASE-FDPASS", run_fdpasstest, 0);
-       register_torture_op("BASE-LOCK1",  torture_locktest1,  0);
-       register_torture_op("BASE-LOCK2",  torture_locktest2,  0);
-       register_torture_op("BASE-LOCK3",  torture_locktest3,  0);
-       register_torture_op("BASE-LOCK4",  torture_locktest4,  0);
-       register_torture_op("BASE-LOCK5",  torture_locktest5,  0);
-       register_torture_op("BASE-LOCK6",  torture_locktest6,  0);
-       register_torture_op("BASE-LOCK7",  torture_locktest7,  0);
-       register_torture_op("BASE-UNLINK", torture_unlinktest, 0);
-       register_torture_op("BASE-ATTR",   run_attrtest,   0);
-       register_torture_op("BASE-TRANS2", run_trans2test, 0);
-       register_torture_op("BASE-NEGNOWAIT", run_negprot_nowait, 0);
-       register_torture_op("BASE-DIR1",  torture_dirtest1, 0);
-       register_torture_op("BASE-DIR2",  torture_dirtest2, 0);
-       register_torture_op("BASE-DENY1",  torture_denytest1, 0);
-       register_torture_op("BASE-DENY2",  torture_denytest2, 0);
-       register_torture_op("BASE-DENY3",  torture_denytest3, 0);
-       register_torture_op("BASE-DENYDOS",  torture_denydos_sharing, 0);
-       register_torture_op("BASE-NTDENY1",  NULL, torture_ntdenytest1);
-       register_torture_op("BASE-NTDENY2",  torture_ntdenytest2, 0);
-       register_torture_op("BASE-TCON",  run_tcon_test, 0);
-       register_torture_op("BASE-TCONDEV",  run_tcon_devtype_test, 0);
-       register_torture_op("BASE-VUID", run_vuidtest, 0);
-       register_torture_op("BASE-RW1",  run_readwritetest, 0);
-       register_torture_op("BASE-OPEN", run_opentest, 0);
-       register_torture_op("BASE-DEFER_OPEN", NULL, run_deferopen);
-       register_torture_op("BASE-XCOPY", run_xcopy, 0);
-       register_torture_op("BASE-IOMETER", run_iometer, 0);
-       register_torture_op("BASE-RENAME", torture_test_rename, 0);
-       register_torture_op("BASE-DELETE", torture_test_delete, 0);
-       register_torture_op("BASE-PROPERTIES", torture_test_properties, 0);
-       register_torture_op("BASE-MANGLE", torture_mangle, 0);
-       register_torture_op("BASE-OPENATTR", torture_openattrtest, 0);
-       register_torture_op("BASE-CHARSET", torture_charset, 0);
-       register_torture_op("BASE-CHKPATH",  torture_chkpath_test, 0);
-       register_torture_op("BASE-SECLEAK",  torture_sec_leak, 0);
-       register_torture_op("BASE-DISCONNECT",  torture_disconnect, 0);
-       register_torture_op("BASE-DELAYWRITE", torture_delay_write, 0);
-       register_torture_op("BASE-SAMBA3ERROR", torture_samba3_errorpaths, 0);
-
-       register_torture_op("SCAN-CASETABLE", torture_casetable, 0);
-       register_torture_op("SCAN-UTABLE", torture_utable, 0);
-       register_torture_op("SCAN-SMB", torture_smb_scan, 0);
-       register_torture_op("SCAN-ALIASES", torture_trans2_aliases, 0);
-       register_torture_op("SCAN-TRANS2", torture_trans2_scan, 0);
-       register_torture_op("SCAN-NTTRANS", torture_nttrans_scan, 0);
+       struct torture_suite *suite = torture_suite_create(talloc_autofree_context(), "BASE");
+
+       torture_suite_add_2smb_test(suite, "FDPASS", run_fdpasstest);
+       torture_suite_add_suite(suite, torture_base_locktest(suite));
+       torture_suite_add_1smb_test(suite, "UNLINK", torture_unlinktest);
+       torture_suite_add_1smb_test(suite, "ATTR",   run_attrtest);
+       torture_suite_add_1smb_test(suite, "TRANS2", run_trans2test);
+       torture_suite_add_simple_test(suite, "NEGNOWAIT", run_negprot_nowait);
+       torture_suite_add_1smb_test(suite, "DIR1",  torture_dirtest1);
+       torture_suite_add_1smb_test(suite, "DIR2",  torture_dirtest2);
+       torture_suite_add_1smb_test(suite, "DENY1",  torture_denytest1);
+       torture_suite_add_2smb_test(suite, "DENY2",  torture_denytest2);
+       torture_suite_add_2smb_test(suite, "DENY3",  torture_denytest3);
+       torture_suite_add_1smb_test(suite, "DENYDOS",  torture_denydos_sharing);
+       torture_suite_add_smb_multi_test(suite, "NTDENY1", torture_ntdenytest1);
+       torture_suite_add_2smb_test(suite, "NTDENY2",  torture_ntdenytest2);
+       torture_suite_add_1smb_test(suite, "TCON",  run_tcon_test);
+       torture_suite_add_1smb_test(suite, "TCONDEV",  run_tcon_devtype_test);
+       torture_suite_add_1smb_test(suite, "VUID", run_vuidtest);
+       torture_suite_add_2smb_test(suite, "RW1",  run_readwritetest);
+       torture_suite_add_2smb_test(suite, "OPEN", run_opentest);
+       torture_suite_add_smb_multi_test(suite, "DEFER_OPEN", run_deferopen);
+       torture_suite_add_1smb_test(suite, "XCOPY", run_xcopy);
+       torture_suite_add_1smb_test(suite, "IOMETER", run_iometer);
+       torture_suite_add_1smb_test(suite, "RENAME", torture_test_rename);
+       torture_suite_add_suite(suite, torture_test_delete());
+       torture_suite_add_1smb_test(suite, "PROPERTIES", torture_test_properties);
+       torture_suite_add_1smb_test(suite, "MANGLE", torture_mangle);
+       torture_suite_add_1smb_test(suite, "OPENATTR", torture_openattrtest);
+       torture_suite_add_suite(suite, torture_charset(suite));
+       torture_suite_add_1smb_test(suite, "CHKPATH",  torture_chkpath_test);
+       torture_suite_add_1smb_test(suite, "SECLEAK",  torture_sec_leak);
+       torture_suite_add_simple_test(suite, "DISCONNECT",  torture_disconnect);
+       torture_suite_add_suite(suite, torture_delay_write());
+       torture_suite_add_simple_test(suite, "SAMBA3ERROR", torture_samba3_errorpaths);
+       torture_suite_add_1smb_test(suite, "CASETABLE", torture_casetable);
+       torture_suite_add_1smb_test(suite, "UTABLE", torture_utable);
+       torture_suite_add_simple_test(suite, "SMB", torture_smb_scan);
+       torture_suite_add_suite(suite, torture_trans2_aliases(suite));
+       torture_suite_add_1smb_test(suite, "TRANS2-SCAN", torture_trans2_scan);
+       torture_suite_add_1smb_test(suite, "NTTRANS", torture_nttrans_scan);
+
+       torture_suite_add_simple_test(suite, "BENCH-HOLDCON", torture_holdcon);
+       torture_suite_add_simple_test(suite, "BENCH-READWRITE", run_benchrw);
+       torture_suite_add_smb_multi_test(suite, "BENCH-TORTURE", run_torture);
+       torture_suite_add_1smb_test(suite, "SCAN-PIPE_NUMBER", run_pipe_number);
+       torture_suite_add_1smb_test(suite, "SCAN-IOCTL", torture_ioctl_test);
+       torture_suite_add_smb_multi_test(suite, "SCAN-MAXFID", run_maxfidtest);
+
+       suite->description = talloc_strdup(suite, 
+                                       "Basic SMB tests (imported from the original smbtorture)");
+
+       torture_register_suite(suite);
 
        return NT_STATUS_OK;
 }