ntvfs/cifs: fix the fnum on RAW_RENAME_NTTRANS
[jelmer/samba4-debian.git] / source / ntvfs / cifs / vfs_cifs.c
index 2741e97d2da44a3bcd2c85b614648f1f79ff7415..3c090b5f5c65a9991709a45c9ca3884c493aa978 100644 (file)
@@ -8,7 +8,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,
@@ -17,8 +17,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/>.
 */
 /*
   this implements a CIFS->CIFS NTVFS filesystem backend. 
 #include "includes.h"
 #include "libcli/raw/libcliraw.h"
 #include "libcli/smb_composite/smb_composite.h"
-#include "smb_server/smb_server.h"
-#include "smbd/service_stream.h"
 #include "auth/auth.h"
+#include "auth/credentials/credentials.h"
 #include "ntvfs/ntvfs.h"
-#include "include/dlinklist.h"
+#include "lib/util/dlinklist.h"
+#include "param/param.h"
+#include "libcli/resolve/resolve.h"
+
+struct cvfs_file {
+       struct cvfs_file *prev, *next;
+       uint16_t fnum;
+       struct ntvfs_handle *h;
+};
 
 /* this is stored in ntvfs_private */
 struct cvfs_private {
        struct smbcli_tree *tree;
        struct smbcli_transport *transport;
-       struct smbsrv_tcon *tcon;
+       struct ntvfs_module_context *ntvfs;
        struct async_info *pending;
-       BOOL map_generic;
+       struct cvfs_file *files;
+       bool map_generic;
+       bool map_trans2;
 };
 
 
@@ -50,21 +58,67 @@ struct async_info {
        struct cvfs_private *cvfs;
        struct ntvfs_request *req;
        struct smbcli_request *c_req;
+       struct cvfs_file *f;
        void *parms;
 };
 
-#define SETUP_PID private->tree->session->pid = SVAL(req->in.hdr, HDR_PID)
+#define SETUP_PID private->tree->session->pid = req->smbpid
+
+#define SETUP_FILE_HERE(f) do { \
+       f = ntvfs_handle_get_backend_data(io->generic.in.file.ntvfs, ntvfs); \
+       if (!f) return NT_STATUS_INVALID_HANDLE; \
+       io->generic.in.file.fnum = f->fnum; \
+} while (0)
+
+#define SETUP_FILE do { \
+       struct cvfs_file *f; \
+       SETUP_FILE_HERE(f); \
+} while (0)
+
+#define SETUP_PID_AND_FILE do { \
+       SETUP_PID; \
+       SETUP_FILE; \
+} while (0)
+
+#define CIFS_SERVER            "cifs:server"
+#define CIFS_USER              "cifs:user"
+#define CIFS_PASSWORD          "cifs:password"
+#define CIFS_DOMAIN            "cifs:domain"
+#define CIFS_SHARE             "cifs:share"
+#define CIFS_USE_MACHINE_ACCT  "cifs:use-machine-account"
+#define CIFS_MAP_GENERIC       "cifs:map-generic"
+#define CIFS_MAP_TRANS2                "cifs:map-trans2"
+
+#define CIFS_USE_MACHINE_ACCT_DEFAULT  false
+#define CIFS_MAP_GENERIC_DEFAULT       false
+#define CIFS_MAP_TRANS2_DEFAULT                true
 
 /*
   a handler for oplock break events from the server - these need to be passed
   along to the client
  */
-static BOOL oplock_handler(struct smbcli_transport *transport, uint16_t tid, uint16_t fnum, uint8_t level, void *p_private)
+static bool oplock_handler(struct smbcli_transport *transport, uint16_t tid, uint16_t fnum, uint8_t level, void *p_private)
 {
        struct cvfs_private *private = p_private;
-       
+       NTSTATUS status;
+       struct ntvfs_handle *h = NULL;
+       struct cvfs_file *f;
+
+       for (f=private->files; f; f=f->next) {
+               if (f->fnum != fnum) continue;
+               h = f->h;
+               break;
+       }
+
+       if (!h) {
+               DEBUG(5,("vfs_cifs: ignoring oplock break level %d for fnum %d\n", level, fnum));
+               return true;
+       }
+
        DEBUG(5,("vfs_cifs: sending oplock break level %d for fnum %d\n", level, fnum));
-       return req_send_oplock_break(private->tcon, fnum, level);
+       status = ntvfs_send_oplock_break(private->ntvfs, h, level);
+       if (!NT_STATUS_IS_OK(status)) return false;
+       return true;
 }
 
 /*
@@ -73,30 +127,30 @@ static BOOL oplock_handler(struct smbcli_transport *transport, uint16_t tid, uin
 static NTSTATUS cvfs_connect(struct ntvfs_module_context *ntvfs, 
                             struct ntvfs_request *req, const char *sharename)
 {
-       struct smbsrv_tcon *tcon = req->tcon;
        NTSTATUS status;
        struct cvfs_private *private;
        const char *host, *user, *pass, *domain, *remote_share;
        struct smb_composite_connect io;
        struct composite_context *creq;
+       struct share_config *scfg = ntvfs->ctx->config;
 
        struct cli_credentials *credentials;
-       BOOL machine_account;
+       bool machine_account;
 
        /* Here we need to determine which server to connect to.
         * For now we use parametric options, type cifs.
         * Later we will use security=server and auth_server.c.
         */
-       host = lp_parm_string(req->tcon->service, "cifs", "server");
-       user = lp_parm_string(req->tcon->service, "cifs", "user");
-       pass = lp_parm_string(req->tcon->service, "cifs", "password");
-       domain = lp_parm_string(req->tcon->service, "cifs", "domain");
-       remote_share = lp_parm_string(req->tcon->service, "cifs", "share");
+       host = share_string_option(scfg, CIFS_SERVER, NULL);
+       user = share_string_option(scfg, CIFS_USER, NULL);
+       pass = share_string_option(scfg, CIFS_PASSWORD, NULL);
+       domain = share_string_option(scfg, CIFS_DOMAIN, NULL);
+       remote_share = share_string_option(scfg, CIFS_SHARE, NULL);
        if (!remote_share) {
                remote_share = sharename;
        }
 
-       machine_account = lp_parm_bool(req->tcon->service, "cifs", "use_machine_account", False);
+       machine_account = share_bool_option(scfg, CIFS_USE_MACHINE_ACCT, CIFS_USE_MACHINE_ACCT_DEFAULT);
 
        private = talloc_zero(ntvfs, struct cvfs_private);
        if (!private) {
@@ -116,7 +170,8 @@ static NTSTATUS cvfs_connect(struct ntvfs_module_context *ntvfs,
                if (!credentials) {
                        return NT_STATUS_NO_MEMORY;
                }
-               cli_credentials_set_conf(credentials);
+               cli_credentials_set_event_context(credentials, ntvfs->ctx->event_ctx);
+               cli_credentials_set_conf(credentials, ntvfs->ctx->lp_ctx);
                cli_credentials_set_username(credentials, user, CRED_SPECIFIED);
                if (domain) {
                        cli_credentials_set_domain(credentials, domain, CRED_SPECIFIED);
@@ -125,33 +180,41 @@ static NTSTATUS cvfs_connect(struct ntvfs_module_context *ntvfs,
        } else if (machine_account) {
                DEBUG(5, ("CIFS backend: Using machine account\n"));
                credentials = cli_credentials_init(private);
-               cli_credentials_set_conf(credentials);
+               cli_credentials_set_event_context(credentials, ntvfs->ctx->event_ctx);
+               cli_credentials_set_conf(credentials, ntvfs->ctx->lp_ctx);
                if (domain) {
                        cli_credentials_set_domain(credentials, domain, CRED_SPECIFIED);
                }
-               status = cli_credentials_set_machine_account(credentials);
+               status = cli_credentials_set_machine_account(credentials, ntvfs->ctx->lp_ctx);
                if (!NT_STATUS_IS_OK(status)) {
                        return status;
                }
-       } else if (req->session->session_info->credentials) {
+       } else if (req->session_info->credentials) {
                DEBUG(5, ("CIFS backend: Using delegated credentials\n"));
-               credentials = req->session->session_info->credentials;
+               credentials = req->session_info->credentials;
        } else {
-               DEBUG(1,("CIFS backend: You must supply server, user and password and or have delegated credentials\n"));
+               DEBUG(1,("CIFS backend: NO delegated credentials found: You must supply server, user and password or the client must supply delegated credentials\n"));
                return NT_STATUS_INVALID_PARAMETER;
        }
 
        /* connect to the server, using the smbd event context */
        io.in.dest_host = host;
-       io.in.port = 0;
+       io.in.dest_ports = lp_smb_ports(ntvfs->ctx->lp_ctx);
        io.in.called_name = host;
        io.in.credentials = credentials;
-       io.in.fallback_to_anonymous = False;
-       io.in.workgroup = lp_workgroup();
+       io.in.fallback_to_anonymous = false;
+       io.in.workgroup = lp_workgroup(ntvfs->ctx->lp_ctx);
        io.in.service = remote_share;
        io.in.service_type = "?????";
-       
-       creq = smb_composite_connect_send(&io, private, tcon->smb_conn->connection->event.ctx);
+       lp_smbcli_options(ntvfs->ctx->lp_ctx, &io.in.options);
+
+       if (!(ntvfs->ctx->client_caps & NTVFS_CLIENT_CAP_LEVEL_II_OPLOCKS)) {
+               io.in.options.use_level2_oplocks = false;
+       }
+
+       creq = smb_composite_connect_send(&io, private, 
+                                         lp_resolve_context(ntvfs->ctx->lp_ctx),
+                                         ntvfs->ctx->event_ctx);
        status = smb_composite_connect_recv(creq, private);
        NT_STATUS_NOT_OK_RETURN(status);
 
@@ -159,16 +222,19 @@ static NTSTATUS cvfs_connect(struct ntvfs_module_context *ntvfs,
 
        private->transport = private->tree->session->transport;
        SETUP_PID;
-       private->tcon = req->tcon;
+       private->ntvfs = ntvfs;
+
+       ntvfs->ctx->fs_type = talloc_strdup(ntvfs->ctx, "NTFS");
+       NT_STATUS_HAVE_NO_MEMORY(ntvfs->ctx->fs_type);
+       ntvfs->ctx->dev_type = talloc_strdup(ntvfs->ctx, "A:");
+       NT_STATUS_HAVE_NO_MEMORY(ntvfs->ctx->dev_type);
 
-       tcon->fs_type = talloc_strdup(tcon, "NTFS");
-       tcon->dev_type = talloc_strdup(tcon, "A:");
-       
        /* we need to receive oplock break requests from the server */
        smbcli_oplock_handler(private->transport, oplock_handler, private);
 
-       private->map_generic = lp_parm_bool(req->tcon->service, 
-                                           "cifs", "mapgeneric", False);
+       private->map_generic = share_bool_option(scfg, CIFS_MAP_GENERIC, CIFS_MAP_GENERIC_DEFAULT);
+
+       private->map_trans2 = share_bool_option(scfg, CIFS_MAP_TRANS2, CIFS_MAP_TRANS2_DEFAULT);
 
        return NT_STATUS_OK;
 }
@@ -179,6 +245,14 @@ static NTSTATUS cvfs_connect(struct ntvfs_module_context *ntvfs,
 static NTSTATUS cvfs_disconnect(struct ntvfs_module_context *ntvfs)
 {
        struct cvfs_private *private = ntvfs->private_data;
+       struct async_info *a, *an;
+
+       /* first cleanup pending requests */
+       for (a=private->pending; a; a = an) {
+               an = a->next;
+               smbcli_request_destroy(a->c_req);
+               talloc_free(a);
+       }
 
        talloc_free(private);
        ntvfs->private_data = NULL;
@@ -189,9 +263,8 @@ static NTSTATUS cvfs_disconnect(struct ntvfs_module_context *ntvfs)
 /*
   destroy an async info structure
 */
-static int async_info_destructor(void *p)
+static int async_info_destructor(struct async_info *async)
 {
-       struct async_info *async = talloc_get_type(p, struct async_info);
        DLIST_REMOVE(async->cvfs->pending, async);
        return 0;
 }
@@ -206,12 +279,13 @@ static void async_simple(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smbcli_request_simple_recv(c_req);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
 
 /* save some typing for the simple functions */
-#define ASYNC_RECV_TAIL(io, async_fn) do { \
+#define ASYNC_RECV_TAIL_F(io, async_fn, file) do { \
        if (!c_req) return NT_STATUS_UNSUCCESSFUL; \
        { \
                struct async_info *async; \
@@ -219,6 +293,7 @@ static void async_simple(struct smbcli_request *c_req)
                if (!async) return NT_STATUS_NO_MEMORY; \
                async->parms = io; \
                async->req = req; \
+               async->f = file; \
                async->cvfs = private; \
                async->c_req = c_req; \
                DLIST_ADD(private->pending, async); \
@@ -230,6 +305,8 @@ static void async_simple(struct smbcli_request *c_req)
        return NT_STATUS_OK; \
 } while (0)
 
+#define ASYNC_RECV_TAIL(io, async_fn) ASYNC_RECV_TAIL_F(io, async_fn, NULL)
+
 #define SIMPLE_ASYNC_TAIL ASYNC_RECV_TAIL(NULL, async_simple)
 
 /*
@@ -263,6 +340,7 @@ static void async_ioctl(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_ioctl_recv(c_req, req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
@@ -270,12 +348,12 @@ static void async_ioctl(struct smbcli_request *c_req)
   ioctl interface
 */
 static NTSTATUS cvfs_ioctl(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_ioctl *io)
+                          struct ntvfs_request *req, union smb_ioctl *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
-       SETUP_PID;
+       SETUP_PID_AND_FILE;
 
        /* see if the front end will allow us to perform this
           function asynchronously.  */
@@ -292,7 +370,7 @@ static NTSTATUS cvfs_ioctl(struct ntvfs_module_context *ntvfs,
   check if a directory exists
 */
 static NTSTATUS cvfs_chkpath(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_chkpath *cp)
+                            struct ntvfs_request *req, union smb_chkpath *cp)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
@@ -316,6 +394,7 @@ static void async_qpathinfo(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_pathinfo_recv(c_req, req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
@@ -323,7 +402,7 @@ static void async_qpathinfo(struct smbcli_request *c_req)
   return info on a pathname
 */
 static NTSTATUS cvfs_qpathinfo(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_fileinfo *info)
+                              struct ntvfs_request *req, union smb_fileinfo *info)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
@@ -347,6 +426,7 @@ static void async_qfileinfo(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_fileinfo_recv(c_req, req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
@@ -354,20 +434,20 @@ static void async_qfileinfo(struct smbcli_request *c_req)
   query info on a open file
 */
 static NTSTATUS cvfs_qfileinfo(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_fileinfo *info)
+                              struct ntvfs_request *req, union smb_fileinfo *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
-       SETUP_PID;
+       SETUP_PID_AND_FILE;
 
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
-               return smb_raw_fileinfo(private->tree, req, info);
+               return smb_raw_fileinfo(private->tree, req, io);
        }
 
-       c_req = smb_raw_fileinfo_send(private->tree, info);
+       c_req = smb_raw_fileinfo_send(private->tree, io);
 
-       ASYNC_RECV_TAIL(info, async_qfileinfo);
+       ASYNC_RECV_TAIL(io, async_qfileinfo);
 }
 
 
@@ -375,7 +455,7 @@ static NTSTATUS cvfs_qfileinfo(struct ntvfs_module_context *ntvfs,
   set info on a pathname
 */
 static NTSTATUS cvfs_setpathinfo(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_setfileinfo *st)
+                                struct ntvfs_request *req, union smb_setfileinfo *st)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
@@ -398,8 +478,22 @@ static NTSTATUS cvfs_setpathinfo(struct ntvfs_module_context *ntvfs,
 static void async_open(struct smbcli_request *c_req)
 {
        struct async_info *async = c_req->async.private;
+       struct cvfs_private *cvfs = async->cvfs;
        struct ntvfs_request *req = async->req;
-       req->async_states->status = smb_raw_open_recv(c_req, req, async->parms);
+       struct cvfs_file *f = async->f;
+       union smb_open *io = async->parms;
+       union smb_handle *file;
+       talloc_free(async);
+       req->async_states->status = smb_raw_open_recv(c_req, req, io);
+       SMB_OPEN_OUT_FILE(io, file);
+       f->fnum = file->fnum;
+       file->ntvfs = NULL;
+       if (!NT_STATUS_IS_OK(req->async_states->status)) goto failed;
+       req->async_states->status = ntvfs_handle_set_backend_data(f->h, cvfs->ntvfs, f);
+       if (!NT_STATUS_IS_OK(req->async_states->status)) goto failed;
+       file->ntvfs = f->h;
+       DLIST_ADD(cvfs->files, f);
+failed:
        req->async_states->send_fn(req);
 }
 
@@ -407,10 +501,13 @@ static void async_open(struct smbcli_request *c_req)
   open a file
 */
 static NTSTATUS cvfs_open(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_open *io)
+                         struct ntvfs_request *req, union smb_open *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
+       struct ntvfs_handle *h;
+       struct cvfs_file *f;
+       NTSTATUS status;
 
        SETUP_PID;
 
@@ -419,20 +516,40 @@ static NTSTATUS cvfs_open(struct ntvfs_module_context *ntvfs,
                return ntvfs_map_open(ntvfs, req, io);
        }
 
+       status = ntvfs_handle_new(ntvfs, req, &h);
+       NT_STATUS_NOT_OK_RETURN(status);
+
+       f = talloc_zero(h, struct cvfs_file);
+       NT_STATUS_HAVE_NO_MEMORY(f);
+       f->h = h;
+
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
-               return smb_raw_open(private->tree, req, io);
+               union smb_handle *file;
+
+               status = smb_raw_open(private->tree, req, io);
+               NT_STATUS_NOT_OK_RETURN(status);
+
+               SMB_OPEN_OUT_FILE(io, file);
+               f->fnum = file->fnum;
+               file->ntvfs = NULL;
+               status = ntvfs_handle_set_backend_data(f->h, private->ntvfs, f);
+               NT_STATUS_NOT_OK_RETURN(status);
+               file->ntvfs = f->h;
+               DLIST_ADD(private->files, f);
+
+               return NT_STATUS_OK;
        }
 
        c_req = smb_raw_open_send(private->tree, io);
 
-       ASYNC_RECV_TAIL(io, async_open);
+       ASYNC_RECV_TAIL_F(io, async_open, f);
 }
 
 /*
   create a directory
 */
 static NTSTATUS cvfs_mkdir(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_mkdir *md)
+                          struct ntvfs_request *req, union smb_mkdir *md)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
@@ -452,7 +569,7 @@ static NTSTATUS cvfs_mkdir(struct ntvfs_module_context *ntvfs,
   remove a directory
 */
 static NTSTATUS cvfs_rmdir(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, struct smb_rmdir *rd)
+                          struct ntvfs_request *req, struct smb_rmdir *rd)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
@@ -471,13 +588,20 @@ static NTSTATUS cvfs_rmdir(struct ntvfs_module_context *ntvfs,
   rename a set of files
 */
 static NTSTATUS cvfs_rename(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_rename *ren)
+                           struct ntvfs_request *req, union smb_rename *ren)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
        SETUP_PID;
 
+       if (ren->nttrans.level == RAW_RENAME_NTTRANS) {
+               struct cvfs_file *f;
+               f = ntvfs_handle_get_backend_data(ren->nttrans.in.file.ntvfs, ntvfs);
+               if (!f) return NT_STATUS_INVALID_HANDLE;
+               ren->nttrans.in.file.fnum = f->fnum;
+       }
+
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
                return smb_raw_rename(private->tree, ren);
        }
@@ -491,7 +615,7 @@ static NTSTATUS cvfs_rename(struct ntvfs_module_context *ntvfs,
   copy a set of files
 */
 static NTSTATUS cvfs_copy(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, struct smb_copy *cp)
+                         struct ntvfs_request *req, struct smb_copy *cp)
 {
        return NT_STATUS_NOT_SUPPORTED;
 }
@@ -504,6 +628,7 @@ static void async_read(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_read_recv(c_req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
@@ -511,25 +636,27 @@ static void async_read(struct smbcli_request *c_req)
   read from a file
 */
 static NTSTATUS cvfs_read(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_read *rd)
+                         struct ntvfs_request *req, union smb_read *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
        SETUP_PID;
 
-       if (rd->generic.level != RAW_READ_GENERIC &&
+       if (io->generic.level != RAW_READ_GENERIC &&
            private->map_generic) {
-               return ntvfs_map_read(ntvfs, req, rd);
+               return ntvfs_map_read(ntvfs, req, io);
        }
 
+       SETUP_FILE;
+
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
-               return smb_raw_read(private->tree, rd);
+               return smb_raw_read(private->tree, io);
        }
 
-       c_req = smb_raw_read_send(private->tree, rd);
+       c_req = smb_raw_read_send(private->tree, io);
 
-       ASYNC_RECV_TAIL(rd, async_read);
+       ASYNC_RECV_TAIL(io, async_read);
 }
 
 /*
@@ -540,6 +667,7 @@ static void async_write(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_write_recv(c_req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
@@ -547,25 +675,26 @@ static void async_write(struct smbcli_request *c_req)
   write to a file
 */
 static NTSTATUS cvfs_write(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_write *wr)
+                          struct ntvfs_request *req, union smb_write *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
        SETUP_PID;
 
-       if (wr->generic.level != RAW_WRITE_GENERIC &&
+       if (io->generic.level != RAW_WRITE_GENERIC &&
            private->map_generic) {
-               return ntvfs_map_write(ntvfs, req, wr);
+               return ntvfs_map_write(ntvfs, req, io);
        }
+       SETUP_FILE;
 
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
-               return smb_raw_write(private->tree, wr);
+               return smb_raw_write(private->tree, io);
        }
 
-       c_req = smb_raw_write_send(private->tree, wr);
+       c_req = smb_raw_write_send(private->tree, io);
 
-       ASYNC_RECV_TAIL(wr, async_write);
+       ASYNC_RECV_TAIL(io, async_write);
 }
 
 /*
@@ -576,6 +705,7 @@ static void async_seek(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_seek_recv(c_req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
@@ -589,7 +719,7 @@ static NTSTATUS cvfs_seek(struct ntvfs_module_context *ntvfs,
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
-       SETUP_PID;
+       SETUP_PID_AND_FILE;
 
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
                return smb_raw_seek(private->tree, io);
@@ -611,6 +741,16 @@ static NTSTATUS cvfs_flush(struct ntvfs_module_context *ntvfs,
        struct smbcli_request *c_req;
 
        SETUP_PID;
+       switch (io->generic.level) {
+       case RAW_FLUSH_FLUSH:
+               SETUP_FILE;
+               break;
+       case RAW_FLUSH_ALL:
+               io->generic.in.file.fnum = 0xFFFF;
+               break;
+       case RAW_FLUSH_SMB2:
+               return NT_STATUS_INVALID_LEVEL;
+       }
 
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
                return smb_raw_flush(private->tree, io);
@@ -625,10 +765,11 @@ static NTSTATUS cvfs_flush(struct ntvfs_module_context *ntvfs,
   close a file
 */
 static NTSTATUS cvfs_close(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_close *io)
+                          struct ntvfs_request *req, union smb_close *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
+       struct cvfs_file *f;
 
        SETUP_PID;
 
@@ -636,6 +777,12 @@ static NTSTATUS cvfs_close(struct ntvfs_module_context *ntvfs,
            private->map_generic) {
                return ntvfs_map_close(ntvfs, req, io);
        }
+       SETUP_FILE_HERE(f);
+       /* Note, we aren't free-ing f, or it's h here. Should we?
+          even if file-close fails, we'll remove it from the list,
+          what else would we do? Maybe we should not remove until
+          after the proxied call completes? */
+       DLIST_REMOVE(private->files, f);
 
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
                return smb_raw_close(private->tree, io);
@@ -650,7 +797,7 @@ static NTSTATUS cvfs_close(struct ntvfs_module_context *ntvfs,
   exit - closing files open by the pid
 */
 static NTSTATUS cvfs_exit(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req)
+                         struct ntvfs_request *req)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
@@ -670,7 +817,7 @@ static NTSTATUS cvfs_exit(struct ntvfs_module_context *ntvfs,
   logoff - closing files open by the user
 */
 static NTSTATUS cvfs_logoff(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req)
+                           struct ntvfs_request *req)
 {
        /* we can't do this right in the cifs backend .... */
        return NT_STATUS_OK;
@@ -693,11 +840,11 @@ static NTSTATUS cvfs_cancel(struct ntvfs_module_context *ntvfs,
                            struct ntvfs_request *req)
 {
        struct cvfs_private *private = ntvfs->private_data;
+       struct async_info *a;
 
        /* find the matching request */
-       struct async_info *a;
        for (a=private->pending;a;a=a->next) {
-               if (SVAL(a->req->in.hdr, HDR_MID) == SVAL(req->in.hdr, HDR_MID)) {
+               if (a->req == req) {
                        break;
                }
        }
@@ -713,23 +860,24 @@ static NTSTATUS cvfs_cancel(struct ntvfs_module_context *ntvfs,
   lock a byte range
 */
 static NTSTATUS cvfs_lock(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_lock *lck)
+                         struct ntvfs_request *req, union smb_lock *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
        SETUP_PID;
 
-       if (lck->generic.level != RAW_LOCK_GENERIC &&
+       if (io->generic.level != RAW_LOCK_GENERIC &&
            private->map_generic) {
-               return ntvfs_map_lock(ntvfs, req, lck);
+               return ntvfs_map_lock(ntvfs, req, io);
        }
+       SETUP_FILE;
 
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
-               return smb_raw_lock(private->tree, lck);
+               return smb_raw_lock(private->tree, io);
        }
 
-       c_req = smb_raw_lock_send(private->tree, lck);
+       c_req = smb_raw_lock_send(private->tree, io);
        SIMPLE_ASYNC_TAIL;
 }
 
@@ -738,17 +886,17 @@ static NTSTATUS cvfs_lock(struct ntvfs_module_context *ntvfs,
 */
 static NTSTATUS cvfs_setfileinfo(struct ntvfs_module_context *ntvfs, 
                                 struct ntvfs_request *req, 
-                                union smb_setfileinfo *info)
+                                union smb_setfileinfo *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
-       SETUP_PID;
+       SETUP_PID_AND_FILE;
 
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
-               return smb_raw_setfileinfo(private->tree, info);
+               return smb_raw_setfileinfo(private->tree, io);
        }
-       c_req = smb_raw_setfileinfo_send(private->tree, info);
+       c_req = smb_raw_setfileinfo_send(private->tree, io);
 
        SIMPLE_ASYNC_TAIL;
 }
@@ -762,6 +910,7 @@ static void async_fsinfo(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_fsinfo_recv(c_req, req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
@@ -769,7 +918,7 @@ static void async_fsinfo(struct smbcli_request *c_req)
   return filesystem space info
 */
 static NTSTATUS cvfs_fsinfo(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_fsinfo *fs)
+                           struct ntvfs_request *req, union smb_fsinfo *fs)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
@@ -789,7 +938,7 @@ static NTSTATUS cvfs_fsinfo(struct ntvfs_module_context *ntvfs,
   return print queue info
 */
 static NTSTATUS cvfs_lpq(struct ntvfs_module_context *ntvfs, 
-                               struct ntvfs_request *req, union smb_lpq *lpq)
+                        struct ntvfs_request *req, union smb_lpq *lpq)
 {
        return NT_STATUS_NOT_SUPPORTED;
 }
@@ -800,7 +949,7 @@ static NTSTATUS cvfs_lpq(struct ntvfs_module_context *ntvfs,
 static NTSTATUS cvfs_search_first(struct ntvfs_module_context *ntvfs, 
                                  struct ntvfs_request *req, union smb_search_first *io, 
                                  void *search_private, 
-                                 BOOL (*callback)(void *, union smb_search_data *))
+                                 bool (*callback)(void *, const union smb_search_data *))
 {
        struct cvfs_private *private = ntvfs->private_data;
 
@@ -813,7 +962,7 @@ static NTSTATUS cvfs_search_first(struct ntvfs_module_context *ntvfs,
 static NTSTATUS cvfs_search_next(struct ntvfs_module_context *ntvfs, 
                                 struct ntvfs_request *req, union smb_search_next *io, 
                                 void *search_private, 
-                                BOOL (*callback)(void *, union smb_search_data *))
+                                bool (*callback)(void *, const union smb_search_data *))
 {
        struct cvfs_private *private = ntvfs->private_data;
 
@@ -841,6 +990,7 @@ static void async_trans2(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_trans2_recv(c_req, req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
@@ -852,6 +1002,10 @@ static NTSTATUS cvfs_trans2(struct ntvfs_module_context *ntvfs,
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
 
+       if (private->map_trans2) {
+               return NT_STATUS_NOT_IMPLEMENTED;
+       }
+
        SETUP_PID;
 
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
@@ -880,27 +1034,44 @@ static void async_changenotify(struct smbcli_request *c_req)
        struct async_info *async = c_req->async.private;
        struct ntvfs_request *req = async->req;
        req->async_states->status = smb_raw_changenotify_recv(c_req, req, async->parms);
+       talloc_free(async);
        req->async_states->send_fn(req);
 }
 
 /* change notify request - always async */
 static NTSTATUS cvfs_notify(struct ntvfs_module_context *ntvfs, 
                            struct ntvfs_request *req,
-                           union smb_notify *info)
+                           union smb_notify *io)
 {
        struct cvfs_private *private = ntvfs->private_data;
        struct smbcli_request *c_req;
+       int saved_timeout = private->transport->options.request_timeout;
+       struct cvfs_file *f;
+
+       if (io->nttrans.level != RAW_NOTIFY_NTTRANS) {
+               return NT_STATUS_NOT_IMPLEMENTED;
+       }
 
        SETUP_PID;
 
+       f = ntvfs_handle_get_backend_data(io->nttrans.in.file.ntvfs, ntvfs);
+       if (!f) return NT_STATUS_INVALID_HANDLE;
+       io->nttrans.in.file.fnum = f->fnum;
+
        /* this request doesn't make sense unless its async */
        if (!(req->async_states->state & NTVFS_ASYNC_STATE_MAY_ASYNC)) {
                return NT_STATUS_INVALID_PARAMETER;
        }
-       
-       c_req = smb_raw_changenotify_send(private->tree, info);
 
-       ASYNC_RECV_TAIL(info, async_changenotify);
+       /* we must not timeout on notify requests - they wait
+          forever */
+       private->transport->options.request_timeout = 0;
+
+       c_req = smb_raw_changenotify_send(private->tree, io);
+
+       private->transport->options.request_timeout = saved_timeout;
+
+       ASYNC_RECV_TAIL(io, async_changenotify);
 }
 
 /*
@@ -910,6 +1081,7 @@ NTSTATUS ntvfs_cifs_init(void)
 {
        NTSTATUS ret;
        struct ntvfs_ops ops;
+       NTVFS_CURRENT_CRITICAL_SIZES(vers);
 
        ZERO_STRUCT(ops);
 
@@ -949,14 +1121,11 @@ NTSTATUS ntvfs_cifs_init(void)
        ops.async_setup = cvfs_async_setup;
        ops.cancel = cvfs_cancel;
        ops.notify = cvfs_notify;
-
-       if (lp_parm_bool(-1, "cifs", "maptrans2", False)) {
-               ops.trans2 = cvfs_trans2;
-       }
+       ops.trans2 = cvfs_trans2;
 
        /* register ourselves with the NTVFS subsystem. We register
           under the name 'cifs'. */
-       ret = ntvfs_register(&ops);
+       ret = ntvfs_register(&ops, &vers);
 
        if (!NT_STATUS_IS_OK(ret)) {
                DEBUG(0,("Failed to register CIFS backend!\n"));