r3323: more warning reductions
[bbaumbach/samba-autobuild/.git] / source4 / ntvfs / simple / vfs_simple.c
index 013419e4eea357a92bd4135b4e3eb8fbe646639f..379d24f8485a872b553cd3d9acbabc5cb2f1cb6e 100644 (file)
@@ -61,19 +61,19 @@ static ssize_t pwrite(int __fd, const void *__buf, size_t __nbytes, off_t __offs
   directory exists (tho it doesn't need to be accessible by the user,
   that comes later)
 */
-static NTSTATUS svfs_connect(struct request_context *req, const char *sharename)
+static NTSTATUS svfs_connect(struct ntvfs_module_context *ntvfs,
+                            struct smbsrv_request *req, const char *sharename)
 {
        struct stat st;
        struct smbsrv_tcon *tcon = req->tcon;
        struct svfs_private *private;
 
-       tcon->ntvfs_private = talloc_p(tcon->mem_ctx, struct svfs_private);
-
-       private = tcon->ntvfs_private;
+       private = talloc_p(tcon, struct svfs_private);
 
        private->next_search_handle = 0;
-       private->connectpath = talloc_strdup(tcon->mem_ctx, lp_pathname(tcon->service));
+       private->connectpath = talloc_strdup(tcon, lp_pathname(tcon->service));
        private->open_files = NULL;
+       private->search = NULL;
 
        /* the directory must exist */
        if (stat(private->connectpath, &st) != 0 || !S_ISDIR(st.st_mode)) {
@@ -82,8 +82,10 @@ static NTSTATUS svfs_connect(struct request_context *req, const char *sharename)
                return NT_STATUS_BAD_NETWORK_NAME;
        }
 
-       tcon->fs_type = talloc_strdup(tcon->mem_ctx, "NTFS");
-       tcon->dev_type = talloc_strdup(tcon->mem_ctx, "A:");
+       tcon->fs_type = talloc_strdup(tcon, "NTFS");
+       tcon->dev_type = talloc_strdup(tcon, "A:");
+
+       ntvfs->private_data = private;
 
        DEBUG(0,("WARNING: ntvfs simple: connect to share [%s] with ROOT privileges!!!\n",sharename));
 
@@ -93,7 +95,8 @@ static NTSTATUS svfs_connect(struct request_context *req, const char *sharename)
 /*
   disconnect from a share
 */
-static NTSTATUS svfs_disconnect(struct smbsrv_tcon *tcon)
+static NTSTATUS svfs_disconnect(struct ntvfs_module_context *ntvfs,
+                               struct smbsrv_tcon *tcon)
 {
        return NT_STATUS_OK;
 }
@@ -116,13 +119,14 @@ static struct svfs_file *find_fd(struct svfs_private *private, int fd)
   delete a file - the dirtype specifies the file types to include in the search. 
   The name can contain CIFS wildcards, but rarely does (except with OS/2 clients)
 */
-static NTSTATUS svfs_unlink(struct request_context *req, struct smb_unlink *unl)
+static NTSTATUS svfs_unlink(struct ntvfs_module_context *ntvfs,
+                           struct smbsrv_request *req, struct smb_unlink *unl)
 {
        char *unix_path;
 
        CHECK_READ_ONLY(req);
 
-       unix_path = svfs_unix_path(req, unl->in.pattern);
+       unix_path = svfs_unix_path(ntvfs, req, unl->in.pattern);
 
        /* ignoring wildcards ... */
        if (unlink(unix_path) == -1) {
@@ -136,7 +140,8 @@ static NTSTATUS svfs_unlink(struct request_context *req, struct smb_unlink *unl)
 /*
   ioctl interface - we don't do any
 */
-static NTSTATUS svfs_ioctl(struct request_context *req, union smb_ioctl *io)
+static NTSTATUS svfs_ioctl(struct ntvfs_module_context *ntvfs,
+                          struct smbsrv_request *req, union smb_ioctl *io)
 {
        return NT_STATUS_INVALID_PARAMETER;
 }
@@ -144,12 +149,13 @@ static NTSTATUS svfs_ioctl(struct request_context *req, union smb_ioctl *io)
 /*
   check if a directory exists
 */
-static NTSTATUS svfs_chkpath(struct request_context *req, struct smb_chkpath *cp)
+static NTSTATUS svfs_chkpath(struct ntvfs_module_context *ntvfs,
+                            struct smbsrv_request *req, struct smb_chkpath *cp)
 {
        char *unix_path;
        struct stat st;
 
-       unix_path = svfs_unix_path(req, cp->in.path);
+       unix_path = svfs_unix_path(ntvfs, req, cp->in.path);
 
        if (stat(unix_path, &st) == -1) {
                return map_nt_error_from_unix(errno);
@@ -165,7 +171,8 @@ static NTSTATUS svfs_chkpath(struct request_context *req, struct smb_chkpath *cp
 /*
   approximately map a struct stat to a generic fileinfo struct
 */
-static NTSTATUS svfs_map_fileinfo(struct request_context *req, union smb_fileinfo *info, 
+static NTSTATUS svfs_map_fileinfo(struct ntvfs_module_context *ntvfs,
+                                 struct smbsrv_request *req, union smb_fileinfo *info, 
                                  struct stat *st, const char *unix_path)
 {
        struct svfs_dir *dir = NULL;
@@ -183,7 +190,7 @@ static NTSTATUS svfs_map_fileinfo(struct request_context *req, union smb_fileinf
        asprintf(&pattern, "%s:*", unix_path);
        
        if (pattern) {
-               dir = svfs_list_unix(req->mem_ctx, req, pattern);
+               dir = svfs_list_unix(req, req, pattern);
        }
 
        unix_to_nt_time(&info->generic.out.create_time, st->st_ctime);
@@ -201,9 +208,8 @@ static NTSTATUS svfs_map_fileinfo(struct request_context *req, union smb_fileinf
        info->generic.out.delete_pending = 0;
        info->generic.out.ea_size = 0;
        info->generic.out.num_eas = 0;
-       info->generic.out.fname.s = talloc_strdup(req->mem_ctx, short_name);
-       info->generic.out.alt_fname.s = talloc_strdup(req->mem_ctx, short_name);
-       info->generic.out.ex_attrib = 0;
+       info->generic.out.fname.s = talloc_strdup(req, short_name);
+       info->generic.out.alt_fname.s = talloc_strdup(req, short_name);
        info->generic.out.compressed_size = 0;
        info->generic.out.format = 0;
        info->generic.out.unit_shift = 0;
@@ -218,7 +224,7 @@ static NTSTATUS svfs_map_fileinfo(struct request_context *req, union smb_fileinf
        info->generic.out.num_streams = 0;
        /* setup a single data stream */
        info->generic.out.num_streams = 1 + (dir?dir->count:0);
-       info->generic.out.streams = talloc_array_p(req->mem_ctx
+       info->generic.out.streams = talloc_array_p(req, 
                                                   struct stream_struct,
                                                   info->generic.out.num_streams);
        if (!info->generic.out.streams) {
@@ -226,7 +232,7 @@ static NTSTATUS svfs_map_fileinfo(struct request_context *req, union smb_fileinf
        }
        info->generic.out.streams[0].size = st->st_size;
        info->generic.out.streams[0].alloc_size = st->st_size;
-       info->generic.out.streams[0].stream_name.s = talloc_strdup(req->mem_ctx,"::$DATA");
+       info->generic.out.streams[0].stream_name.s = talloc_strdup(req,"::$DATA");
 
        for (i=0;dir && i<dir->count;i++) {
                s = strchr(dir->files[i].name, ':');
@@ -241,39 +247,39 @@ static NTSTATUS svfs_map_fileinfo(struct request_context *req, union smb_fileinf
 /*
   return info on a pathname
 */
-static NTSTATUS svfs_qpathinfo(struct request_context *req, union smb_fileinfo *info)
+static NTSTATUS svfs_qpathinfo(struct ntvfs_module_context *ntvfs,
+                              struct smbsrv_request *req, union smb_fileinfo *info)
 {
        char *unix_path;
        struct stat st;
 
        DEBUG(19,("svfs_qpathinfo: file %s level 0x%x\n", info->generic.in.fname, info->generic.level));
        if (info->generic.level != RAW_FILEINFO_GENERIC) {
-               return ntvfs_map_qpathinfo(req, info);
+               return ntvfs_map_qpathinfo(req, info, ntvfs);
        }
        
-       unix_path = svfs_unix_path(req, info->generic.in.fname);
+       unix_path = svfs_unix_path(ntvfs, req, info->generic.in.fname);
        DEBUG(19,("svfs_qpathinfo: file %s\n", unix_path));
        if (stat(unix_path, &st) == -1) {
                DEBUG(19,("svfs_qpathinfo: file %s errno=%d\n", unix_path, errno));
-               if (errno == 0)
-                       errno = ENOENT;
                return map_nt_error_from_unix(errno);
        }
        DEBUG(19,("svfs_qpathinfo: file %s, stat done\n", unix_path));
-       return svfs_map_fileinfo(req, info, &st, unix_path);
+       return svfs_map_fileinfo(ntvfs, req, info, &st, unix_path);
 }
 
 /*
   query info on a open file
 */
-static NTSTATUS svfs_qfileinfo(struct request_context *req, union smb_fileinfo *info)
+static NTSTATUS svfs_qfileinfo(struct ntvfs_module_context *ntvfs,
+                              struct smbsrv_request *req, union smb_fileinfo *info)
 {
-       struct svfs_private *private = req->tcon->ntvfs_private;
+       struct svfs_private *private = ntvfs->private_data;
        struct svfs_file *f;
        struct stat st;
 
        if (info->generic.level != RAW_FILEINFO_GENERIC) {
-               return ntvfs_map_qfileinfo(req, info);
+               return ntvfs_map_qfileinfo(req, info, ntvfs);
        }
 
        f = find_fd(private, info->generic.in.fnum);
@@ -282,21 +288,20 @@ static NTSTATUS svfs_qfileinfo(struct request_context *req, union smb_fileinfo *
        }
        
        if (fstat(info->generic.in.fnum, &st) == -1) {
-               if (errno == 0)
-                       errno = ENOENT;
                return map_nt_error_from_unix(errno);
        }
 
-       return svfs_map_fileinfo(req, info, &st, f->name);
+       return svfs_map_fileinfo(ntvfs, req,info, &st, f->name);
 }
 
 
 /*
   open a file
 */
-static NTSTATUS svfs_open(struct request_context *req, union smb_open *io)
+static NTSTATUS svfs_open(struct ntvfs_module_context *ntvfs,
+                         struct smbsrv_request *req, union smb_open *io)
 {
-       struct svfs_private *private = req->tcon->ntvfs_private;
+       struct svfs_private *private = ntvfs->private_data;
        char *unix_path;
        struct stat st;
        int fd, flags;
@@ -304,7 +309,7 @@ static NTSTATUS svfs_open(struct request_context *req, union smb_open *io)
        int create_flags, rdwr_flags;
        
        if (io->generic.level != RAW_OPEN_GENERIC) {
-               return ntvfs_map_open(req, io);
+               return ntvfs_map_open(req, io, ntvfs);
        }
 
        if (lp_readonly(req->tcon->service)) {
@@ -315,7 +320,7 @@ static NTSTATUS svfs_open(struct request_context *req, union smb_open *io)
                rdwr_flags = O_RDWR;
        }
 
-       unix_path = svfs_unix_path(req, io->ntcreatex.in.fname);
+       unix_path = svfs_unix_path(ntvfs, req, io->ntcreatex.in.fname);
 
        switch (io->generic.in.open_disposition) {
        case NTCREATEX_DISP_SUPERSEDE:
@@ -363,22 +368,18 @@ static NTSTATUS svfs_open(struct request_context *req, union smb_open *io)
 do_open:
        fd = open(unix_path, flags, 0644);
        if (fd == -1) {
-               if (errno == 0)
-                       errno = ENOENT;
                return map_nt_error_from_unix(errno);
        }
 
        if (fstat(fd, &st) == -1) {
                DEBUG(9,("svfs_open: fstat errno=%d\n", errno));
-               if (errno == 0)
-                       errno = ENOENT;
                close(fd);
                return map_nt_error_from_unix(errno);
        }
 
-       f = talloc_p(req->tcon->mem_ctx, struct svfs_file);
+       f = talloc_p(req->tcon, struct svfs_file);
        f->fd = fd;
-       f->name = talloc_strdup(req->tcon->mem_ctx, unix_path);
+       f->name = talloc_strdup(req->tcon, unix_path);
 
        DLIST_ADD(private->open_files, f);
 
@@ -400,7 +401,8 @@ do_open:
 /*
   create a directory
 */
-static NTSTATUS svfs_mkdir(struct request_context *req, union smb_mkdir *md)
+static NTSTATUS svfs_mkdir(struct ntvfs_module_context *ntvfs,
+                          struct smbsrv_request *req, union smb_mkdir *md)
 {
        char *unix_path;
 
@@ -410,7 +412,7 @@ static NTSTATUS svfs_mkdir(struct request_context *req, union smb_mkdir *md)
                return NT_STATUS_INVALID_LEVEL;
        }
 
-       unix_path = svfs_unix_path(req, md->mkdir.in.path);
+       unix_path = svfs_unix_path(ntvfs, req, md->mkdir.in.path);
 
        if (mkdir(unix_path, 0777) == -1) {
                return map_nt_error_from_unix(errno);
@@ -422,13 +424,14 @@ static NTSTATUS svfs_mkdir(struct request_context *req, union smb_mkdir *md)
 /*
   remove a directory
 */
-static NTSTATUS svfs_rmdir(struct request_context *req, struct smb_rmdir *rd)
+static NTSTATUS svfs_rmdir(struct ntvfs_module_context *ntvfs,
+                          struct smbsrv_request *req, struct smb_rmdir *rd)
 {
        char *unix_path;
 
        CHECK_READ_ONLY(req);
 
-       unix_path = svfs_unix_path(req, rd->in.path);
+       unix_path = svfs_unix_path(ntvfs, req, rd->in.path);
 
        if (rmdir(unix_path) == -1) {
                return map_nt_error_from_unix(errno);
@@ -440,7 +443,8 @@ static NTSTATUS svfs_rmdir(struct request_context *req, struct smb_rmdir *rd)
 /*
   rename a set of files
 */
-static NTSTATUS svfs_rename(struct request_context *req, union smb_rename *ren)
+static NTSTATUS svfs_rename(struct ntvfs_module_context *ntvfs,
+                           struct smbsrv_request *req, union smb_rename *ren)
 {
        char *unix_path1, *unix_path2;
 
@@ -450,10 +454,10 @@ static NTSTATUS svfs_rename(struct request_context *req, union smb_rename *ren)
                return NT_STATUS_INVALID_LEVEL;
        }
 
-       unix_path1 = svfs_unix_path(req, ren->rename.in.pattern1);
-       unix_path2 = svfs_unix_path(req, ren->rename.in.pattern2);
+       unix_path1 = svfs_unix_path(ntvfs, req, ren->rename.in.pattern1);
+       unix_path2 = svfs_unix_path(ntvfs, req, ren->rename.in.pattern2);
 
-       if (rename(unix_path1, unix_path2) != 0) {
+       if (rename(unix_path1, unix_path2) == -1) {
                return map_nt_error_from_unix(errno);
        }
        
@@ -463,7 +467,8 @@ static NTSTATUS svfs_rename(struct request_context *req, union smb_rename *ren)
 /*
   copy a set of files
 */
-static NTSTATUS svfs_copy(struct request_context *req, struct smb_copy *cp)
+static NTSTATUS svfs_copy(struct ntvfs_module_context *ntvfs,
+                         struct smbsrv_request *req, struct smb_copy *cp)
 {
        return NT_STATUS_NOT_SUPPORTED;
 }
@@ -471,7 +476,8 @@ static NTSTATUS svfs_copy(struct request_context *req, struct smb_copy *cp)
 /*
   read from a file
 */
-static NTSTATUS svfs_read(struct request_context *req, union smb_read *rd)
+static NTSTATUS svfs_read(struct ntvfs_module_context *ntvfs,
+                         struct smbsrv_request *req, union smb_read *rd)
 {
        ssize_t ret;
 
@@ -497,53 +503,36 @@ static NTSTATUS svfs_read(struct request_context *req, union smb_read *rd)
 /*
   write to a file
 */
-static NTSTATUS svfs_write(struct request_context *req, union smb_write *wr)
+static NTSTATUS svfs_write(struct ntvfs_module_context *ntvfs,
+                          struct smbsrv_request *req, union smb_write *wr)
 {
        ssize_t ret;
 
-       CHECK_READ_ONLY(req);
+       if (wr->generic.level != RAW_WRITE_WRITEX) {
+               return ntvfs_map_write(req, wr, ntvfs);
+       }
 
-       switch (wr->generic.level) {
-       case RAW_WRITE_WRITEX:
-               ret = pwrite(wr->writex.in.fnum, 
-                            wr->writex.in.data, 
-                            wr->writex.in.count,
-                            wr->writex.in.offset);
-               if (ret == -1) {
-                       return map_nt_error_from_unix(errno);
-               }
-               
-               wr->writex.out.nwritten = ret;
-               wr->writex.out.remaining = 0; /* should fill this in? */
-
-               return NT_STATUS_OK;
-
-       case RAW_WRITE_WRITE:
-               if (wr->write.in.count == 0) {
-                       /* a truncate! */
-                       ret = ftruncate(wr->write.in.fnum, wr->write.in.offset);
-               } else {
-                       ret = pwrite(wr->write.in.fnum, 
-                                    wr->write.in.data, 
-                                    wr->write.in.count,
-                                    wr->write.in.offset);
-               }
-               if (ret == -1) {
-                       return map_nt_error_from_unix(errno);
-               }
-               
-               wr->write.out.nwritten = ret;
+       CHECK_READ_ONLY(req);
 
-               return NT_STATUS_OK;
+       ret = pwrite(wr->writex.in.fnum, 
+                    wr->writex.in.data, 
+                    wr->writex.in.count,
+                    wr->writex.in.offset);
+       if (ret == -1) {
+               return map_nt_error_from_unix(errno);
        }
-
-       return NT_STATUS_NOT_SUPPORTED;
+               
+       wr->writex.out.nwritten = ret;
+       wr->writex.out.remaining = 0; /* should fill this in? */
+       
+       return NT_STATUS_OK;
 }
 
 /*
   seek in a file
 */
-static NTSTATUS svfs_seek(struct request_context *req, struct smb_seek *io)
+static NTSTATUS svfs_seek(struct ntvfs_module_context *ntvfs,
+                         struct smbsrv_request *req, struct smb_seek *io)
 {
        return NT_STATUS_NOT_SUPPORTED;
 }
@@ -551,7 +540,8 @@ static NTSTATUS svfs_seek(struct request_context *req, struct smb_seek *io)
 /*
   flush a file
 */
-static NTSTATUS svfs_flush(struct request_context *req, struct smb_flush *io)
+static NTSTATUS svfs_flush(struct ntvfs_module_context *ntvfs,
+                          struct smbsrv_request *req, struct smb_flush *io)
 {
        fsync(io->in.fnum);
        return NT_STATUS_OK;
@@ -560,9 +550,10 @@ static NTSTATUS svfs_flush(struct request_context *req, struct smb_flush *io)
 /*
   close a file
 */
-static NTSTATUS svfs_close(struct request_context *req, union smb_close *io)
+static NTSTATUS svfs_close(struct ntvfs_module_context *ntvfs,
+                          struct smbsrv_request *req, union smb_close *io)
 {
-       struct svfs_private *private = req->tcon->ntvfs_private;
+       struct svfs_private *private = ntvfs->private_data;
        struct svfs_file *f;
 
        if (io->generic.level != RAW_CLOSE_CLOSE) {
@@ -575,29 +566,50 @@ static NTSTATUS svfs_close(struct request_context *req, union smb_close *io)
                return NT_STATUS_INVALID_HANDLE;
        }
 
-       if (close(io->close.in.fnum) != 0) {
+       if (close(io->close.in.fnum) == -1) {
                return map_nt_error_from_unix(errno);
        }
 
        DLIST_REMOVE(private->open_files, f);
-       talloc_free(req->tcon->mem_ctx, f->name);
-       talloc_free(req->tcon->mem_ctx, f);
+       talloc_free(f->name);
+       talloc_free(f);
 
        return NT_STATUS_OK;
 }
 
 /*
-  exit - closing files?
+  exit - closing files
 */
-static NTSTATUS svfs_exit(struct request_context *req)
+static NTSTATUS svfs_exit(struct ntvfs_module_context *ntvfs,
+                         struct smbsrv_request *req)
 {
        return NT_STATUS_NOT_SUPPORTED;
 }
 
+/*
+  logoff - closing files
+*/
+static NTSTATUS svfs_logoff(struct ntvfs_module_context *ntvfs,
+                           struct smbsrv_request *req)
+{
+       return NT_STATUS_NOT_SUPPORTED;
+}
+
+/*
+  setup for an async call
+*/
+static NTSTATUS svfs_async_setup(struct ntvfs_module_context *ntvfs,
+                                struct smbsrv_request *req, 
+                                void *private)
+{
+       return NT_STATUS_OK;
+}
+
 /*
   lock a byte range
 */
-static NTSTATUS svfs_lock(struct request_context *req, union smb_lock *lck)
+static NTSTATUS svfs_lock(struct ntvfs_module_context *ntvfs,
+                         struct smbsrv_request *req, union smb_lock *lck)
 {
        DEBUG(0,("REWRITE: not doing byte range locking!\n"));
        return NT_STATUS_OK;
@@ -606,7 +618,8 @@ static NTSTATUS svfs_lock(struct request_context *req, union smb_lock *lck)
 /*
   set info on a pathname
 */
-static NTSTATUS svfs_setpathinfo(struct request_context *req, union smb_setfileinfo *st)
+static NTSTATUS svfs_setpathinfo(struct ntvfs_module_context *ntvfs,
+                                struct smbsrv_request *req, union smb_setfileinfo *st)
 {
        CHECK_READ_ONLY(req);
 
@@ -616,7 +629,8 @@ static NTSTATUS svfs_setpathinfo(struct request_context *req, union smb_setfilei
 /*
   set info on a open file
 */
-static NTSTATUS svfs_setfileinfo(struct request_context *req, 
+static NTSTATUS svfs_setfileinfo(struct ntvfs_module_context *ntvfs,
+                                struct smbsrv_request *req, 
                                 union smb_setfileinfo *info)
 {
        struct utimbuf unix_times;
@@ -628,7 +642,7 @@ static NTSTATUS svfs_setfileinfo(struct request_context *req,
        case RAW_SFILEINFO_END_OF_FILE_INFO:
        case RAW_SFILEINFO_END_OF_FILE_INFORMATION:
                if (ftruncate(info->end_of_file_info.file.fnum, 
-                             info->end_of_file_info.in.size) != 0) {
+                             info->end_of_file_info.in.size) == -1) {
                        return map_nt_error_from_unix(errno);
                }
                break;
@@ -659,13 +673,14 @@ static NTSTATUS svfs_setfileinfo(struct request_context *req,
 /*
   return filesystem space info
 */
-static NTSTATUS svfs_fsinfo(struct request_context *req, union smb_fsinfo *fs)
+static NTSTATUS svfs_fsinfo(struct ntvfs_module_context *ntvfs,
+                           struct smbsrv_request *req, union smb_fsinfo *fs)
 {
-       struct svfs_private *private = req->tcon->ntvfs_private;
+       struct svfs_private *private = ntvfs->private_data;
        struct stat st;
 
        if (fs->generic.level != RAW_QFS_GENERIC) {
-               return ntvfs_map_fsinfo(req, fs);
+               return ntvfs_map_fsinfo(req, fs, ntvfs);
        }
 
        if (sys_fsusage(private->connectpath, 
@@ -690,7 +705,7 @@ static NTSTATUS svfs_fsinfo(struct request_context *req, union smb_fsinfo *fs)
        fs->generic.out.quota_soft = 0;
        fs->generic.out.quota_hard = 0;
        fs->generic.out.quota_flags = 0;
-       fs->generic.out.volume_name = talloc_strdup(req->mem_ctx, lp_servicename(req->tcon->service));
+       fs->generic.out.volume_name = talloc_strdup(req, lp_servicename(req->tcon->service));
        fs->generic.out.fs_type = req->tcon->fs_type;
 
        return NT_STATUS_OK;
@@ -700,16 +715,17 @@ static NTSTATUS svfs_fsinfo(struct request_context *req, union smb_fsinfo *fs)
 /*
   return filesystem attribute info
 */
-static NTSTATUS svfs_fsattr(struct request_context *req, union smb_fsattr *fs)
+static NTSTATUS svfs_fsattr(struct ntvfs_module_context *ntvfs,
+                           struct smbsrv_request *req, union smb_fsattr *fs)
 {
        struct stat st;
-       struct svfs_private *private = req->tcon->ntvfs_private;
+       struct svfs_private *private = ntvfs->private_data;
 
        if (fs->generic.level != RAW_FSATTR_GENERIC) {
-               return ntvfs_map_fsattr(req, fs);
+               return ntvfs_map_fsattr(req, fs, ntvfs);
        }
 
-       if (stat(private->connectpath, &st) != 0) {
+       if (stat(private->connectpath, &st) == -1) {
                return map_nt_error_from_unix(errno);
        }
 
@@ -720,8 +736,8 @@ static NTSTATUS svfs_fsattr(struct request_context *req, union smb_fsattr *fs)
                FILE_PERSISTENT_ACLS;
        fs->generic.out.max_file_component_length = 255;
        fs->generic.out.serial_number = 1;
-       fs->generic.out.fs_type = talloc_strdup(req->mem_ctx, "NTFS");
-       fs->generic.out.volume_name = talloc_strdup(req->mem_ctx
+       fs->generic.out.fs_type = talloc_strdup(req, "NTFS");
+       fs->generic.out.volume_name = talloc_strdup(req, 
                                                    lp_servicename(req->tcon->service));
 
        return NT_STATUS_OK;
@@ -731,7 +747,8 @@ static NTSTATUS svfs_fsattr(struct request_context *req, union smb_fsattr *fs)
 /*
   return print queue info
 */
-static NTSTATUS svfs_lpq(struct request_context *req, union smb_lpq *lpq)
+static NTSTATUS svfs_lpq(struct ntvfs_module_context *ntvfs,
+                        struct smbsrv_request *req, union smb_lpq *lpq)
 {
        return NT_STATUS_NOT_SUPPORTED;
 }
@@ -739,38 +756,34 @@ static NTSTATUS svfs_lpq(struct request_context *req, union smb_lpq *lpq)
 /* 
    list files in a directory matching a wildcard pattern
 */
-static NTSTATUS svfs_search_first(struct request_context *req, union smb_search_first *io, 
+static NTSTATUS svfs_search_first(struct ntvfs_module_context *ntvfs,
+                                 struct smbsrv_request *req, union smb_search_first *io, 
                                  void *search_private, 
                                  BOOL (*callback)(void *, union smb_search_data *))
 {
        struct svfs_dir *dir;
        int i;
-       struct svfs_private *private = req->tcon->ntvfs_private;
+       struct svfs_private *private = ntvfs->private_data;
        struct search_state *search;
        union smb_search_data file;
-       TALLOC_CTX *mem_ctx;
        uint_t max_count;
 
        if (io->generic.level != RAW_SEARCH_BOTH_DIRECTORY_INFO) {
                return NT_STATUS_NOT_SUPPORTED;
        }
 
-       mem_ctx = talloc_init("svfs_search");
-
-       search = talloc_zero(mem_ctx, sizeof(struct search_state));
+       search = talloc_zero_p(private, struct search_state);
        if (!search) {
                return NT_STATUS_NO_MEMORY;
        }
 
        max_count = io->t2ffirst.in.max_count;
 
-       dir = svfs_list(mem_ctx, req, io->t2ffirst.in.pattern);
+       dir = svfs_list(ntvfs, req, io->t2ffirst.in.pattern);
        if (!dir) {
-               talloc_destroy_pool(mem_ctx);
                return NT_STATUS_FOOBAR;
        }
 
-       search->mem_ctx = mem_ctx;
        search->handle = private->next_search_handle;
        search->dir = dir;
 
@@ -803,7 +816,7 @@ static NTSTATUS svfs_search_first(struct request_context *req, union smb_search_
        /* work out if we are going to keep the search state */
        if ((io->t2ffirst.in.flags & FLAG_TRANS2_FIND_CLOSE) ||
            ((io->t2ffirst.in.flags & FLAG_TRANS2_FIND_CLOSE_IF_END) && (i == dir->count))) {
-               talloc_destroy(search->mem_ctx);
+               talloc_free(search);
        } else {
                private->next_search_handle++;
                DLIST_ADD(private->search, search);
@@ -813,13 +826,14 @@ static NTSTATUS svfs_search_first(struct request_context *req, union smb_search_
 }
 
 /* continue a search */
-static NTSTATUS svfs_search_next(struct request_context *req, union smb_search_next *io, 
+static NTSTATUS svfs_search_next(struct ntvfs_module_context *ntvfs,
+                                struct smbsrv_request *req, union smb_search_next *io, 
                                 void *search_private, 
                                 BOOL (*callback)(void *, union smb_search_data *))
 {
        struct svfs_dir *dir;
        int i;
-       struct svfs_private *private = req->tcon->ntvfs_private;
+       struct svfs_private *private = ntvfs->private_data;
        struct search_state *search;
        union smb_search_data file;
        uint_t max_count;
@@ -893,16 +907,17 @@ found:
        if ((io->t2fnext.in.flags & FLAG_TRANS2_FIND_CLOSE) ||
            ((io->t2fnext.in.flags & FLAG_TRANS2_FIND_CLOSE_IF_END) && (i == dir->count))) {
                DLIST_REMOVE(private->search, search);
-               talloc_destroy(search->mem_ctx);
+               talloc_free(search);
        }
 
        return NT_STATUS_OK;
 }
 
 /* close a search */
-static NTSTATUS svfs_search_close(struct request_context *req, union smb_search_close *io)
+static NTSTATUS svfs_search_close(struct ntvfs_module_context *ntvfs,
+                                 struct smbsrv_request *req, union smb_search_close *io)
 {
-       struct svfs_private *private = req->tcon->ntvfs_private;
+       struct svfs_private *private = ntvfs->private_data;
        struct search_state *search;
 
        for (search=private->search; search; search = search->next) {
@@ -915,13 +930,14 @@ static NTSTATUS svfs_search_close(struct request_context *req, union smb_search_
        }
 
        DLIST_REMOVE(private->search, search);
-       talloc_destroy(search->mem_ctx);
+       talloc_free(search);
 
        return NT_STATUS_OK;
 }
 
 /* SMBtrans - not used on file shares */
-static NTSTATUS svfs_trans(struct request_context *req, struct smb_trans2 *trans2)
+static NTSTATUS svfs_trans(struct ntvfs_module_context *ntvfs,
+                          struct smbsrv_request *req, struct smb_trans2 *trans2)
 {
        return NT_STATUS_ACCESS_DENIED;
 }
@@ -937,9 +953,6 @@ NTSTATUS ntvfs_simple_init(void)
 
        ZERO_STRUCT(ops);
 
-       /* fill in the name and type */
-       ops.type = NTVFS_DISK;
-
        /* fill in all the operations */
        ops.connect = svfs_connect;
        ops.disconnect = svfs_disconnect;
@@ -968,10 +981,14 @@ NTSTATUS ntvfs_simple_init(void)
        ops.search_next = svfs_search_next;
        ops.search_close = svfs_search_close;
        ops.trans = svfs_trans;
+       ops.logoff = svfs_logoff;
+       ops.async_setup = svfs_async_setup;
 
        /* register ourselves with the NTVFS subsystem. We register
-          under names 'simple' 
+          under names 'simple'
        */
+
+       ops.type = NTVFS_DISK;
        ops.name = "simple";
        ret = register_backend("ntvfs", &ops);