Add an exception for S3
[samba.git] / source4 / torture / raw / streams.c
index 978a5fcc3a9ccdde6dcf7c5fcbfbf0a919bdd17f..ca6b488af5a0dd86f478be5b5d07cbf8f2c20d19 100644 (file)
@@ -7,7 +7,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,
@@ -16,8 +16,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"
@@ -33,7 +32,7 @@
        if (!NT_STATUS_EQUAL(status, correct)) { \
                printf("(%s) Incorrect status %s - should be %s\n", \
                       __location__, nt_errstr(status), nt_errstr(correct)); \
-               ret = False; \
+               ret = false; \
                goto done; \
        }} while (0)
 
        if ((v) != (correct)) { \
                printf("(%s) Incorrect value %s=%d - should be %d\n", \
                       __location__, #v, (int)v, (int)correct); \
-               ret = False; \
+               ret = false; \
        }} while (0)
 
 /*
   check that a stream has the right contents
 */
-static BOOL check_stream(struct smbcli_state *cli, TALLOC_CTX *mem_ctx,
+static bool check_stream(struct smbcli_state *cli, const char *location,
+                        TALLOC_CTX *mem_ctx,
                         const char *fname, const char *sname, 
                         const char *value)
 {
@@ -62,53 +62,136 @@ static BOOL check_stream(struct smbcli_state *cli, TALLOC_CTX *mem_ctx,
 
        if (value == NULL) {
                if (fnum != -1) {
-                       printf("should have failed stream open of %s\n", full_name);
-                       return False;
+                       printf("(%s) should have failed stream open of %s\n",
+                              location, full_name);
+                       return false;
                }
-               return True;
+               return true;
        }
            
        if (fnum == -1) {
-               printf("Failed to open stream '%s' - %s\n", 
-                      full_name, smbcli_errstr(cli->tree));
-               return False;
+               printf("(%s) Failed to open stream '%s' - %s\n",
+                      location, full_name, smbcli_errstr(cli->tree));
+               return false;
        }
 
-       buf = talloc_size(mem_ctx, strlen(value)+11);
+       buf = talloc_array(mem_ctx, uint8_t, strlen(value)+11);
        
        ret = smbcli_read(cli->tree, fnum, buf, 0, strlen(value)+11);
        if (ret != strlen(value)) {
-               printf("Failed to read %lu bytes from stream '%s' - got %d\n",
-                      (long)strlen(value), full_name, (int)ret);
-               return False;
+               printf("(%s) Failed to read %lu bytes from stream '%s' - got %d\n",
+                      location, (long)strlen(value), full_name, (int)ret);
+               return false;
        }
 
        if (memcmp(buf, value, strlen(value)) != 0) {
-               printf("Bad data in stream\n");
-               return False;
+               printf("(%s) Bad data in stream\n", location);
+               return false;
        }
 
        smbcli_close(cli->tree, fnum);
-       return True;
+       return true;
+}
+
+static int qsort_string(const void *v1, const void *v2)
+{
+       char * const *s1 = v1;
+       char * const *s2 = v2;
+       return strcmp(*s1, *s2);
+}
+
+static int qsort_stream(const void *v1, const void *v2)
+{
+       const struct stream_struct * s1 = v1;
+       const struct stream_struct * s2 = v2;
+       return strcmp(s1->stream_name.s, s2->stream_name.s);
+}
+
+static bool check_stream_list(struct smbcli_state *cli, const char *fname,
+                             int num_exp, const char **exp)
+{
+       union smb_fileinfo finfo;
+       NTSTATUS status;
+       int i;
+       TALLOC_CTX *tmp_ctx = talloc_new(cli);
+       char **exp_sort;
+       struct stream_struct *stream_sort;
+       bool ret = false;
+
+       finfo.generic.level = RAW_FILEINFO_STREAM_INFO;
+       finfo.generic.in.file.path = fname;
+
+       status = smb_raw_pathinfo(cli->tree, tmp_ctx, &finfo);
+       if (!NT_STATUS_IS_OK(status)) {
+               d_fprintf(stderr, "(%s) smb_raw_pathinfo failed: %s\n",
+                         __location__, nt_errstr(status));
+               goto fail;
+       }
+
+       if (finfo.stream_info.out.num_streams != num_exp) {
+               d_fprintf(stderr, "(%s) expected %d streams, got %d\n",
+                         __location__, num_exp,
+                         finfo.stream_info.out.num_streams);
+               goto fail;
+       }
+
+       exp_sort = talloc_memdup(tmp_ctx, exp, num_exp * sizeof(*exp));
+
+       if (exp_sort == NULL) {
+               goto fail;
+       }
+
+       qsort(exp_sort, num_exp, sizeof(*exp_sort), qsort_string);
+
+       stream_sort = talloc_memdup(tmp_ctx, finfo.stream_info.out.streams,
+                                   finfo.stream_info.out.num_streams *
+                                   sizeof(*stream_sort));
+
+       if (stream_sort == NULL) {
+               goto fail;
+       }
+
+       qsort(stream_sort, finfo.stream_info.out.num_streams,
+             sizeof(*stream_sort), qsort_stream);
+
+       for (i=0; i<num_exp; i++) {
+               if (strcmp(exp_sort[i], stream_sort[i].stream_name.s) != 0) {
+                       d_fprintf(stderr, "(%s) expected stream name %s, got "
+                                 "%s\n", __location__, exp_sort[i],
+                                 stream_sort[i].stream_name.s);
+                       goto fail;
+               }
+       }
+
+       ret = true;
+ fail:
+       talloc_free(tmp_ctx);
+       return ret;
 }
 
 /*
   test basic io on streams
 */
-static BOOL test_stream_io(struct smbcli_state *cli, TALLOC_CTX *mem_ctx)
+static bool test_stream_io(struct torture_context *tctx,
+                          struct smbcli_state *cli, TALLOC_CTX *mem_ctx)
 {
        NTSTATUS status;
        union smb_open io;
        const char *fname = BASEDIR "\\stream.txt";
        const char *sname1, *sname2;
-       BOOL ret = True;
+       bool ret = true;
        int fnum = -1;
        ssize_t retsize;
 
+       const char *one[] = { "::$DATA" };
+       const char *two[] = { "::$DATA", ":Second Stream:$DATA" };
+       const char *three[] = { "::$DATA", ":Stream One:$DATA",
+                               ":Second Stream:$DATA" };
+
        sname1 = talloc_asprintf(mem_ctx, "%s:%s", fname, "Stream One");
        sname2 = talloc_asprintf(mem_ctx, "%s:%s:$DaTa", fname, "Second Stream");
 
-       printf("opening non-existant directory stream\n");
+       printf("(%s) opening non-existant directory stream\n", __location__);
        io.generic.level = RAW_OPEN_NTCREATEX;
        io.ntcreatex.in.root_fid = 0;
        io.ntcreatex.in.flags = 0;
@@ -124,29 +207,29 @@ static BOOL test_stream_io(struct smbcli_state *cli, TALLOC_CTX *mem_ctx)
        status = smb_raw_open(cli->tree, mem_ctx, &io);
        CHECK_STATUS(status, NT_STATUS_NOT_A_DIRECTORY);
 
-       printf("creating a stream on a non-existant file\n");
+       printf("(%s) creating a stream on a non-existant file\n", __location__);
        io.ntcreatex.in.create_options = 0;
        io.ntcreatex.in.fname = sname1;
        status = smb_raw_open(cli->tree, mem_ctx, &io);
        CHECK_STATUS(status, NT_STATUS_OK);
        fnum = io.ntcreatex.out.file.fnum;
 
-       ret &= check_stream(cli, mem_ctx, fname, "Stream One", NULL);
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Stream One", NULL);
 
-       printf("check that open of base file is allowed\n");
+       printf("(%s) check that open of base file is allowed\n", __location__);
        io.ntcreatex.in.open_disposition = NTCREATEX_DISP_OPEN;
        io.ntcreatex.in.fname = fname;
        status = smb_raw_open(cli->tree, mem_ctx, &io);
        CHECK_STATUS(status, NT_STATUS_OK);
        smbcli_close(cli->tree, io.ntcreatex.out.file.fnum);
 
-       printf("writing to stream\n");
+       printf("(%s) writing to stream\n", __location__);
        retsize = smbcli_write(cli->tree, fnum, 0, "test data", 0, 9);
        CHECK_VALUE(retsize, 9);
 
        smbcli_close(cli->tree, fnum);
 
-       ret &= check_stream(cli, mem_ctx, fname, "Stream One", "test data");
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Stream One", "test data");
 
        io.ntcreatex.in.open_disposition = NTCREATEX_DISP_OPEN;
        io.ntcreatex.in.fname = sname1;
@@ -154,45 +237,54 @@ static BOOL test_stream_io(struct smbcli_state *cli, TALLOC_CTX *mem_ctx)
        CHECK_STATUS(status, NT_STATUS_OK);
        fnum = io.ntcreatex.out.file.fnum;
 
-       printf("modifying stream\n");
+       printf("(%s) modifying stream\n", __location__);
        retsize = smbcli_write(cli->tree, fnum, 0, "MORE DATA ", 5, 10);
        CHECK_VALUE(retsize, 10);
 
        smbcli_close(cli->tree, fnum);
 
-       ret &= check_stream(cli, mem_ctx, fname, "Stream One:$FOO", NULL);
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Stream One:$FOO", NULL);
 
-       printf("creating a stream2 on a existing file\n");
+       printf("(%s) creating a stream2 on a existing file\n", __location__);
        io.ntcreatex.in.fname = sname2;
        io.ntcreatex.in.open_disposition = NTCREATEX_DISP_OPEN_IF;
        status = smb_raw_open(cli->tree, mem_ctx, &io);
        CHECK_STATUS(status, NT_STATUS_OK);
        fnum = io.ntcreatex.out.file.fnum;
 
-       printf("modifying stream\n");
+       printf("(%s) modifying stream\n", __location__);
        retsize = smbcli_write(cli->tree, fnum, 0, "SECOND STREAM", 0, 13);
        CHECK_VALUE(retsize, 13);
 
        smbcli_close(cli->tree, fnum);
 
-       ret &= check_stream(cli, mem_ctx, fname, "Stream One", "test MORE DATA ");
-       ret &= check_stream(cli, mem_ctx, fname, "Stream One:$DATA", "test MORE DATA ");
-       ret &= check_stream(cli, mem_ctx, fname, "Stream One:", NULL);
-       ret &= check_stream(cli, mem_ctx, fname, "Second Stream", "SECOND STREAM");
-       ret &= check_stream(cli, mem_ctx, fname, "Second Stream:$DATA", "SECOND STREAM");
-       ret &= check_stream(cli, mem_ctx, fname, "Second Stream:", NULL);
-       ret &= check_stream(cli, mem_ctx, fname, "Second Stream:$FOO", NULL);
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Stream One", "test MORE DATA ");
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Stream One:$DATA", "test MORE DATA ");
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Stream One:", NULL);
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Second Stream", "SECOND STREAM");
+       if (!torture_setting_bool(tctx, "samba4", false)) {
+               ret &= check_stream(cli, __location__, mem_ctx, fname,
+                                   "SECOND STREAM:$DATA", "SECOND STREAM");
+       }
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Second Stream:$DATA", "SECOND STREAM");
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Second Stream:", NULL);
+       ret &= check_stream(cli, __location__, mem_ctx, fname, "Second Stream:$FOO", NULL);
 
-       printf("deleting stream\n");
+       check_stream_list(cli, fname, 3, three);
+
+       printf("(%s) deleting stream\n", __location__);
        status = smbcli_unlink(cli->tree, sname1);
        CHECK_STATUS(status, NT_STATUS_OK);
 
-       printf("delete a stream via delete-on-close\n");
+       check_stream_list(cli, fname, 2, two);
+
+       printf("(%s) delete a stream via delete-on-close\n", __location__);
        io.ntcreatex.in.fname = sname2;
        io.ntcreatex.in.create_options = NTCREATEX_OPTIONS_DELETE_ON_CLOSE;
        io.ntcreatex.in.share_access = NTCREATEX_SHARE_ACCESS_DELETE;
        io.ntcreatex.in.access_mask = SEC_RIGHTS_FILE_ALL;
        io.ntcreatex.in.open_disposition = NTCREATEX_DISP_OPEN;
+
        status = smb_raw_open(cli->tree, mem_ctx, &io);
        CHECK_STATUS(status, NT_STATUS_OK);
        fnum = io.ntcreatex.out.file.fnum;
@@ -201,8 +293,21 @@ static BOOL test_stream_io(struct smbcli_state *cli, TALLOC_CTX *mem_ctx)
        status = smbcli_unlink(cli->tree, sname2);
        CHECK_STATUS(status, NT_STATUS_OBJECT_NAME_NOT_FOUND);
 
+       check_stream_list(cli, fname, 1, one);
+
+       if (!torture_setting_bool(tctx, "samba4", false)) {
+               io.ntcreatex.in.open_disposition = NTCREATEX_DISP_CREATE;
+               io.ntcreatex.in.fname = sname1;
+               status = smb_raw_open(cli->tree, mem_ctx, &io);
+               CHECK_STATUS(status, NT_STATUS_OK);
+               smbcli_close(cli->tree, io.ntcreatex.out.file.fnum);
+               io.ntcreatex.in.fname = sname2;
+               status = smb_raw_open(cli->tree, mem_ctx, &io);
+               CHECK_STATUS(status, NT_STATUS_OK);
+               smbcli_close(cli->tree, io.ntcreatex.out.file.fnum);
+       }
 
-       printf("deleting file\n");
+       printf("(%s) deleting file\n", __location__);
        status = smbcli_unlink(cli->tree, fname);
        CHECK_STATUS(status, NT_STATUS_OK);
 
@@ -211,31 +316,255 @@ done:
        return ret;
 }
 
-/* 
-   basic testing of streams calls
+/*
+  test stream sharemodes
 */
-BOOL torture_raw_streams(struct torture_context *torture)
+static bool test_stream_sharemodes(struct torture_context *tctx,
+                                  struct smbcli_state *cli,
+                                  TALLOC_CTX *mem_ctx)
 {
-       struct smbcli_state *cli;
-       BOOL ret = True;
-       TALLOC_CTX *mem_ctx;
+       NTSTATUS status;
+       union smb_open io;
+       const char *fname = BASEDIR "\\stream.txt";
+       const char *sname1, *sname2;
+       bool ret = true;
+       int fnum1 = -1;
+       int fnum2 = -1;
+
+       sname1 = talloc_asprintf(mem_ctx, "%s:%s", fname, "Stream One");
+       sname2 = talloc_asprintf(mem_ctx, "%s:%s:$DaTa", fname, "Second Stream");
+
+       printf("(%s) testing stream share mode conflicts\n", __location__);
+       io.generic.level = RAW_OPEN_NTCREATEX;
+       io.ntcreatex.in.root_fid = 0;
+       io.ntcreatex.in.flags = 0;
+       io.ntcreatex.in.access_mask = SEC_FILE_WRITE_DATA;
+       io.ntcreatex.in.create_options = 0;
+       io.ntcreatex.in.file_attr = FILE_ATTRIBUTE_NORMAL;
+       io.ntcreatex.in.share_access = 0;
+       io.ntcreatex.in.alloc_size = 0;
+       io.ntcreatex.in.open_disposition = NTCREATEX_DISP_CREATE;
+       io.ntcreatex.in.impersonation = NTCREATEX_IMPERSONATION_ANONYMOUS;
+       io.ntcreatex.in.security_flags = 0;
+       io.ntcreatex.in.fname = sname1;
+
+       status = smb_raw_open(cli->tree, mem_ctx, &io);
+       CHECK_STATUS(status, NT_STATUS_OK);
+       fnum1 = io.ntcreatex.out.file.fnum;
+
+       /*
+        * A different stream does not give a sharing violation
+        */
+
+       CHECK_STATUS(status, NT_STATUS_OK);
+       fnum1 = io.ntcreatex.out.file.fnum;
+
+       /*
+        * ... whereas the same stream does with unchanged access/share_access
+        * flags
+        */
+
+       io.ntcreatex.in.open_disposition = 0;
+       status = smb_raw_open(cli->tree, mem_ctx, &io);
+       CHECK_STATUS(status, NT_STATUS_SHARING_VIOLATION);
+
+       io.ntcreatex.in.fname = sname2;
+       status = smb_raw_open(cli->tree, mem_ctx, &io);
+       CHECK_STATUS(status, NT_STATUS_OK);
+       fnum2 = io.ntcreatex.out.file.fnum;
+
+done:
+       if (fnum1 != -1) smbcli_close(cli->tree, fnum1);
+       if (fnum2 != -1) smbcli_close(cli->tree, fnum2);
+       status = smbcli_unlink(cli->tree, fname);
+       return ret;
+}
 
-       if (!torture_open_connection(&cli)) {
-               return False;
+/* 
+ *  Test FILE_SHARE_DELETE on streams
+ *
+ * A stream opened with !FILE_SHARE_DELETE prevents the main file to be opened
+ * with SEC_STD_DELETE.
+ *
+ * The main file opened with !FILE_SHARE_DELETE does *not* prevent a stream to
+ * be opened with SEC_STD_DELETE.
+ *
+ * A stream held open with FILE_SHARE_DELETE allows the file to be
+ * deleted. After the main file is deleted, access to the open file descriptor
+ * still works, but all name-based access to both the main file as well as the
+ * stream is denied with DELETE ending.
+ *
+ * This means, an open of the main file with SEC_STD_DELETE should walk all
+ * streams and also open them with SEC_STD_DELETE. If any of these opens gives
+ * SHARING_VIOLATION, the main open fails.
+ *
+ * Closing the main file after delete_on_close has been set does not really
+ * unlink it but leaves the corresponding share mode entry with
+ * delete_on_close being set around until all streams are closed.
+ *
+ * Opening a stream must also look at the main file's share mode entry, look
+ * at the delete_on_close bit and potentially return DELETE_PENDING.
+ */
+
+static bool test_stream_delete(struct torture_context *tctx,
+                              struct smbcli_state *cli, TALLOC_CTX *mem_ctx)
+{
+       NTSTATUS status;
+       union smb_open io;
+       const char *fname = BASEDIR "\\stream.txt";
+       const char *sname1;
+       bool ret = true;
+       int fnum = -1;
+       uint8_t buf[9];
+       ssize_t retsize;
+       union smb_fileinfo finfo;
+
+       sname1 = talloc_asprintf(mem_ctx, "%s:%s", fname, "Stream One");
+
+       printf("(%s) opening non-existant directory stream\n", __location__);
+       io.generic.level = RAW_OPEN_NTCREATEX;
+       io.ntcreatex.in.root_fid = 0;
+       io.ntcreatex.in.flags = 0;
+       io.ntcreatex.in.access_mask = SEC_FILE_READ_DATA|SEC_FILE_WRITE_DATA;
+       io.ntcreatex.in.create_options = 0;
+       io.ntcreatex.in.file_attr = FILE_ATTRIBUTE_NORMAL;
+       io.ntcreatex.in.share_access = 0;
+       io.ntcreatex.in.alloc_size = 0;
+       io.ntcreatex.in.open_disposition = NTCREATEX_DISP_CREATE;
+       io.ntcreatex.in.impersonation = NTCREATEX_IMPERSONATION_ANONYMOUS;
+       io.ntcreatex.in.security_flags = 0;
+       io.ntcreatex.in.fname = sname1;
+
+       status = smb_raw_open(cli->tree, mem_ctx, &io);
+       CHECK_STATUS(status, NT_STATUS_OK);
+       fnum = io.ntcreatex.out.file.fnum;
+
+       retsize = smbcli_write(cli->tree, fnum, 0, "test data", 0, 9);
+       CHECK_VALUE(retsize, 9);
+
+       /*
+        * One stream opened without FILE_SHARE_DELETE prevents the main file
+        * to be deleted or even opened with DELETE access
+        */
+
+       status = smbcli_unlink(cli->tree, fname);
+       CHECK_STATUS(status, NT_STATUS_SHARING_VIOLATION);
+
+       io.ntcreatex.in.open_disposition = NTCREATEX_DISP_OPEN;
+       io.ntcreatex.in.fname = fname;
+       io.ntcreatex.in.access_mask = SEC_STD_DELETE;
+       status = smb_raw_open(cli->tree, mem_ctx, &io);
+       CHECK_STATUS(status, NT_STATUS_SHARING_VIOLATION);
+
+       smbcli_close(cli->tree, fnum);
+
+       /*
+        * ... but unlink works if a stream is opened with FILE_SHARE_DELETE
+        */
+
+       io.ntcreatex.in.fname = sname1;
+       io.ntcreatex.in.access_mask = SEC_FILE_READ_DATA|SEC_FILE_WRITE_DATA;
+       io.ntcreatex.in.share_access = NTCREATEX_SHARE_ACCESS_DELETE;
+       status = smb_raw_open(cli->tree, mem_ctx, &io);
+       CHECK_STATUS(status, NT_STATUS_OK);
+       fnum = io.ntcreatex.out.file.fnum;
+
+       status = smbcli_unlink(cli->tree, fname);
+       CHECK_STATUS(status, NT_STATUS_OK);
+
+       /*
+        * file access still works on the stream while the main file is closed
+        */
+
+       retsize = smbcli_read(cli->tree, fnum, buf, 0, 9);
+       CHECK_VALUE(retsize, 9);
+
+       finfo.generic.level = RAW_FILEINFO_STANDARD;
+       finfo.generic.in.file.path = fname;
+
+       /*
+        * name-based access to both the main file and the stream does not
+        * work anymore but gives DELETE_PENDING
+        */
+
+       status = smb_raw_pathinfo(cli->tree, mem_ctx, &finfo);
+       CHECK_STATUS(status, NT_STATUS_DELETE_PENDING);
+
+       if (!torture_setting_bool(tctx, "samba3", false)) {
+
+               /*
+                * S3 doesn't do this yet
+                */
+
+               finfo.generic.in.file.path = sname1;
+               status = smb_raw_pathinfo(cli->tree, mem_ctx, &finfo);
+               CHECK_STATUS(status, NT_STATUS_DELETE_PENDING);
        }
 
-       mem_ctx = talloc_init("torture_raw_streams");
+       /*
+        * fd-based qfileinfo on the stream still works, the stream does not
+        * have the delete-on-close bit set. This could mean that open on the
+        * stream first opens the main file
+        */
+
+       finfo.all_info.level = RAW_FILEINFO_ALL_INFO;
+       finfo.all_info.in.file.fnum = fnum;
+
+       status = smb_raw_fileinfo(cli->tree, mem_ctx, &finfo);
+       CHECK_STATUS(status, NT_STATUS_OK);
+       CHECK_VALUE(finfo.all_info.out.delete_pending, 0);
+
+       smbcli_close(cli->tree, fnum);
+
+       /*
+        * After closing the stream the file is really gone.
+        */
+
+       finfo.generic.in.file.path = fname;
+       status = smb_raw_pathinfo(cli->tree, mem_ctx, &finfo);
+       CHECK_STATUS(status, NT_STATUS_OBJECT_NAME_NOT_FOUND);
+
+       io.ntcreatex.in.access_mask = SEC_FILE_READ_DATA|SEC_FILE_WRITE_DATA
+               |SEC_STD_DELETE;
+       io.ntcreatex.in.create_options = NTCREATEX_OPTIONS_DELETE_ON_CLOSE;
+       io.ntcreatex.in.open_disposition = NTCREATEX_DISP_CREATE;
+       status = smb_raw_open(cli->tree, mem_ctx, &io);
+       CHECK_STATUS(status, NT_STATUS_OK);
+       fnum = io.ntcreatex.out.file.fnum;
+
+       finfo.generic.in.file.path = fname;
+       status = smb_raw_pathinfo(cli->tree, mem_ctx, &finfo);
+       CHECK_STATUS(status, NT_STATUS_OK);
+
+       smbcli_close(cli->tree, fnum);
+
+       status = smb_raw_pathinfo(cli->tree, mem_ctx, &finfo);
+       CHECK_STATUS(status, NT_STATUS_OK);
+done:
+       smbcli_close(cli->tree, fnum);
+       return ret;
+}
+
+/* 
+   basic testing of streams calls
+*/
+bool torture_raw_streams(struct torture_context *torture, 
+                        struct smbcli_state *cli)
+{
+       bool ret = true;
 
        if (!torture_setup_dir(cli, BASEDIR)) {
-               return False;
+               return false;
        }
 
-       ret &= test_stream_io(cli, mem_ctx);
+       ret &= test_stream_io(torture, cli, torture);
+       ret &= test_stream_sharemodes(torture, cli, torture);
+       if (!torture_setting_bool(torture, "samba4", false)) {
+               ret &= test_stream_delete(torture, cli, torture);
+       }
 
        smb_raw_exit(cli->session);
        smbcli_deltree(cli->tree, BASEDIR);
 
-       torture_close_connection(cli);
-       talloc_free(mem_ctx);
        return ret;
 }