ldb:utf8: ldb_ascii_toupper() avoids real toupper()
[samba.git] / source3 / modules / vfs_streams_xattr.c
index 631444251525fab363970a6baa92ce42a287d302..03ff6147cb030bd56829670bbdc72f8c22fd5ba4 100644 (file)
@@ -24,7 +24,9 @@
 #include "includes.h"
 #include "smbd/smbd.h"
 #include "system/filesys.h"
-#include "../lib/crypto/md5.h"
+#include "lib/util/tevent_unix.h"
+#include "librpc/gen_ndr/ioctl.h"
+#include "hash_inode.h"
 
 #undef DBGC_CLASS
 #define DBGC_CLASS DBGC_VFS
@@ -43,51 +45,17 @@ struct stream_io {
        vfs_handle_struct *handle;
 };
 
-static SMB_INO_T stream_inode(const SMB_STRUCT_STAT *sbuf, const char *sname)
-{
-       MD5_CTX ctx;
-        unsigned char hash[16];
-       SMB_INO_T result;
-       char *upper_sname;
-
-       DEBUG(10, ("stream_inode called for %lu/%lu [%s]\n",
-                  (unsigned long)sbuf->st_ex_dev,
-                  (unsigned long)sbuf->st_ex_ino, sname));
-
-       upper_sname = talloc_strdup_upper(talloc_tos(), sname);
-       SMB_ASSERT(upper_sname != NULL);
-
-        MD5Init(&ctx);
-        MD5Update(&ctx, (const unsigned char *)&(sbuf->st_ex_dev),
-                 sizeof(sbuf->st_ex_dev));
-        MD5Update(&ctx, (const unsigned char *)&(sbuf->st_ex_ino),
-                 sizeof(sbuf->st_ex_ino));
-        MD5Update(&ctx, (unsigned char *)upper_sname,
-                 talloc_get_size(upper_sname)-1);
-        MD5Final(hash, &ctx);
-
-       TALLOC_FREE(upper_sname);
-
-        /* Hopefully all the variation is in the lower 4 (or 8) bytes! */
-       memcpy(&result, hash, sizeof(result));
-
-       DEBUG(10, ("stream_inode returns %lu\n", (unsigned long)result));
-
-       return result;
-}
-
-static ssize_t get_xattr_size(connection_struct *conn,
-                               files_struct *fsp,
-                               const char *fname,
-                               const char *xattr_name)
+static ssize_t get_xattr_size_fsp(struct files_struct *fsp,
+                                 const char *xattr_name)
 {
        NTSTATUS status;
        struct ea_struct ea;
        ssize_t result;
 
-       status = get_ea_value(talloc_tos(), conn, fsp, fname,
-                             xattr_name, &ea);
-
+       status = get_ea_value_fsp(talloc_tos(),
+                                 fsp,
+                                 xattr_name,
+                                 &ea);
        if (!NT_STATUS_IS_OK(status)) {
                return -1;
        }
@@ -106,36 +74,51 @@ static NTSTATUS streams_xattr_get_name(vfs_handle_struct *handle,
                                       const char *stream_name,
                                       char **xattr_name)
 {
+       size_t stream_name_len = strlen(stream_name);
        char *stype;
        struct streams_xattr_config *config;
 
        SMB_VFS_HANDLE_GET_DATA(handle, config, struct streams_xattr_config,
                                return NT_STATUS_UNSUCCESSFUL);
 
-       stype = strchr_m(stream_name + 1, ':');
-
-       *xattr_name = talloc_asprintf(ctx, "%s%s",
-                                     config->prefix,
-                                     stream_name + 1);
-       if (*xattr_name == NULL) {
-               return NT_STATUS_NO_MEMORY;
-       }
+       SMB_ASSERT(stream_name[0] == ':');
+       stream_name += 1;
 
-       if (stype != NULL) {
-               /* Normalize the stream type to upercase. */
-               if (!strupper_m(strrchr_m(*xattr_name, ':') + 1)) {
-                       return NT_STATUS_INVALID_PARAMETER;
-               }
-       } else if (config->store_stream_type) {
+       /*
+        * With vfs_fruit option "fruit:encoding = native" we're
+        * already converting stream names that contain illegal NTFS
+        * characters from their on-the-wire Unicode Private Range
+        * encoding to their native ASCII representation.
+        *
+        * As as result the name of xattrs storing the streams (via
+        * vfs_streams_xattr) may contain a colon, so we have to use
+        * strrchr_m() instead of strchr_m() for matching the stream
+        * type suffix.
+        *
+        * In check_path_syntax() we've already ensured the streamname
+        * we got from the client is valid.
+        */
+       stype = strrchr_m(stream_name, ':');
+
+       if (stype) {
                /*
-                * Append an explicit stream type if one wasn't
-                * specified.
+                * We only support one stream type: "$DATA"
                 */
-               *xattr_name = talloc_asprintf(ctx, "%s%s",
-                                             *xattr_name, ":$DATA");
-               if (*xattr_name == NULL) {
-                       return NT_STATUS_NO_MEMORY;
+               if (strcasecmp_m(stype, ":$DATA") != 0) {
+                       return NT_STATUS_INVALID_PARAMETER;
                }
+
+               /* Split name and type */
+               stream_name_len = (stype - stream_name);
+       }
+
+       *xattr_name = talloc_asprintf(ctx, "%s%.*s%s",
+                                     config->prefix,
+                                     (int)stream_name_len,
+                                     stream_name,
+                                     config->store_stream_type ? ":$DATA" : "");
+       if (*xattr_name == NULL) {
+               return NT_STATUS_NO_MEMORY;
        }
 
        DEBUG(10, ("xattr_name: %s, stream_name: %s\n", *xattr_name,
@@ -170,88 +153,58 @@ static bool streams_xattr_recheck(struct stream_io *sio)
        TALLOC_FREE(sio->base);
        sio->xattr_name = talloc_strdup(VFS_MEMCTX_FSP_EXTENSION(sio->handle, sio->fsp),
                                        xattr_name);
-       sio->base = talloc_strdup(VFS_MEMCTX_FSP_EXTENSION(sio->handle, sio->fsp),
-                                 sio->fsp->fsp_name->base_name);
-       sio->fsp_name_ptr = sio->fsp->fsp_name;
-
+       if (sio->xattr_name == NULL) {
+               DBG_DEBUG("sio->xattr_name==NULL\n");
+               return false;
+       }
        TALLOC_FREE(xattr_name);
 
-       if ((sio->xattr_name == NULL) || (sio->base == NULL)) {
+       sio->base = talloc_strdup(VFS_MEMCTX_FSP_EXTENSION(sio->handle, sio->fsp),
+                                 sio->fsp->fsp_name->base_name);
+       if (sio->base == NULL) {
+               DBG_DEBUG("sio->base==NULL\n");
                return false;
        }
 
-       return true;
-}
-
-/**
- * Helper to stat/lstat the base file of an smb_fname.
- */
-static int streams_xattr_stat_base(vfs_handle_struct *handle,
-                                  struct smb_filename *smb_fname,
-                                  bool follow_links)
-{
-       char *tmp_stream_name;
-       int result;
+       sio->fsp_name_ptr = sio->fsp->fsp_name;
 
-       tmp_stream_name = smb_fname->stream_name;
-       smb_fname->stream_name = NULL;
-       if (follow_links) {
-               result = SMB_VFS_NEXT_STAT(handle, smb_fname);
-       } else {
-               result = SMB_VFS_NEXT_LSTAT(handle, smb_fname);
-       }
-       smb_fname->stream_name = tmp_stream_name;
-       return result;
+       return true;
 }
 
 static int streams_xattr_fstat(vfs_handle_struct *handle, files_struct *fsp,
                               SMB_STRUCT_STAT *sbuf)
 {
-       struct smb_filename *smb_fname_base = NULL;
        int ret = -1;
        struct stream_io *io = (struct stream_io *)
                VFS_FETCH_FSP_EXTENSION(handle, fsp);
 
-       DEBUG(10, ("streams_xattr_fstat called for %d\n", fsp->fh->fd));
-
-       if (io == NULL || fsp->base_fsp == NULL) {
+       if (io == NULL || !fsp_is_alternate_stream(fsp)) {
                return SMB_VFS_NEXT_FSTAT(handle, fsp, sbuf);
        }
 
-       if (!streams_xattr_recheck(io)) {
-               return -1;
-       }
+       DBG_DEBUG("streams_xattr_fstat called for %s\n", fsp_str_dbg(io->fsp));
 
-       /* Create an smb_filename with stream_name == NULL. */
-       smb_fname_base = synthetic_smb_fname(talloc_tos(), io->base,
-                                            NULL, NULL);
-       if (smb_fname_base == NULL) {
-               errno = ENOMEM;
+       if (!streams_xattr_recheck(io)) {
                return -1;
        }
 
-       if (lp_posix_pathnames()) {
-               ret = SMB_VFS_LSTAT(handle->conn, smb_fname_base);
-       } else {
-               ret = SMB_VFS_STAT(handle->conn, smb_fname_base);
-       }
-       *sbuf = smb_fname_base->st;
-       TALLOC_FREE(smb_fname_base);
-
+       ret = SMB_VFS_NEXT_FSTAT(handle, fsp->base_fsp, sbuf);
        if (ret == -1) {
                return -1;
        }
 
-       sbuf->st_ex_size = get_xattr_size(handle->conn, fsp->base_fsp,
-                                       io->base, io->xattr_name);
+       sbuf->st_ex_size = get_xattr_size_fsp(fsp->base_fsp,
+                                             io->xattr_name);
        if (sbuf->st_ex_size == -1) {
+               SET_STAT_INVALID(*sbuf);
                return -1;
        }
 
        DEBUG(10, ("sbuf->st_ex_size = %d\n", (int)sbuf->st_ex_size));
 
-       sbuf->st_ex_ino = stream_inode(sbuf, io->xattr_name);
+       sbuf->st_ex_ino = hash_inode(sbuf, io->xattr_name);
        sbuf->st_ex_mode &= ~S_IFMT;
+       sbuf->st_ex_mode &= ~S_IFDIR;
         sbuf->st_ex_mode |= S_IFREG;
         sbuf->st_ex_blocks = sbuf->st_ex_size / STAT_ST_BLOCKSIZE + 1;
 
@@ -264,23 +217,26 @@ static int streams_xattr_stat(vfs_handle_struct *handle,
        NTSTATUS status;
        int result = -1;
        char *xattr_name = NULL;
+       char *tmp_stream_name = NULL;
+       struct smb_filename *pathref = NULL;
+       struct files_struct *fsp = smb_fname->fsp;
 
-       if (!is_ntfs_stream_smb_fname(smb_fname)) {
+       if (!is_named_stream(smb_fname)) {
                return SMB_VFS_NEXT_STAT(handle, smb_fname);
        }
 
        /* Note if lp_posix_paths() is true, we can never
-        * get here as is_ntfs_stream_smb_fname() is
+        * get here as is_named_stream() is
         * always false. So we never need worry about
         * not following links here. */
 
-       /* If the default stream is requested, just stat the base file. */
-       if (is_ntfs_default_stream_smb_fname(smb_fname)) {
-               return streams_xattr_stat_base(handle, smb_fname, true);
-       }
-
        /* Populate the stat struct with info from the base file. */
-       if (streams_xattr_stat_base(handle, smb_fname, true) == -1) {
+       tmp_stream_name = smb_fname->stream_name;
+       smb_fname->stream_name = NULL;
+       result = SMB_VFS_NEXT_STAT(handle, smb_fname);
+       smb_fname->stream_name = tmp_stream_name;
+
+       if (result == -1) {
                return -1;
        }
 
@@ -293,113 +249,102 @@ static int streams_xattr_stat(vfs_handle_struct *handle,
        }
 
        /* Augment the base file's stat information before returning. */
-       smb_fname->st.st_ex_size = get_xattr_size(handle->conn, NULL,
-                                                 smb_fname->base_name,
-                                                 xattr_name);
+       if (fsp == NULL) {
+               status = synthetic_pathref(talloc_tos(),
+                                          handle->conn->cwd_fsp,
+                                          smb_fname->base_name,
+                                          NULL,
+                                          NULL,
+                                          smb_fname->twrp,
+                                          smb_fname->flags,
+                                          &pathref);
+               if (!NT_STATUS_IS_OK(status)) {
+                       TALLOC_FREE(xattr_name);
+                       SET_STAT_INVALID(smb_fname->st);
+                       errno = ENOENT;
+                       return -1;
+               }
+               fsp = pathref->fsp;
+       } else {
+               fsp = fsp->base_fsp;
+       }
+
+       smb_fname->st.st_ex_size = get_xattr_size_fsp(fsp,
+                                                     xattr_name);
        if (smb_fname->st.st_ex_size == -1) {
+               TALLOC_FREE(xattr_name);
+               TALLOC_FREE(pathref);
+               SET_STAT_INVALID(smb_fname->st);
                errno = ENOENT;
-               result = -1;
-               goto fail;
+               return -1;
        }
 
-       smb_fname->st.st_ex_ino = stream_inode(&smb_fname->st, xattr_name);
+       smb_fname->st.st_ex_ino = hash_inode(&smb_fname->st, xattr_name);
        smb_fname->st.st_ex_mode &= ~S_IFMT;
         smb_fname->st.st_ex_mode |= S_IFREG;
         smb_fname->st.st_ex_blocks =
            smb_fname->st.st_ex_size / STAT_ST_BLOCKSIZE + 1;
 
-       result = 0;
- fail:
        TALLOC_FREE(xattr_name);
-       return result;
+       TALLOC_FREE(pathref);
+       return 0;
 }
 
 static int streams_xattr_lstat(vfs_handle_struct *handle,
                               struct smb_filename *smb_fname)
 {
-       NTSTATUS status;
-       int result = -1;
-       char *xattr_name = NULL;
-
-       if (!is_ntfs_stream_smb_fname(smb_fname)) {
-               return SMB_VFS_NEXT_LSTAT(handle, smb_fname);
-       }
-
-       /* If the default stream is requested, just stat the base file. */
-       if (is_ntfs_default_stream_smb_fname(smb_fname)) {
-               return streams_xattr_stat_base(handle, smb_fname, false);
-       }
-
-       /* Populate the stat struct with info from the base file. */
-       if (streams_xattr_stat_base(handle, smb_fname, false) == -1) {
-               return -1;
-       }
-
-       /* Derive the xattr name to lookup. */
-       status = streams_xattr_get_name(handle, talloc_tos(),
-                                       smb_fname->stream_name, &xattr_name);
-       if (!NT_STATUS_IS_OK(status)) {
-               errno = map_errno_from_nt_status(status);
-               return -1;
-       }
-
-       /* Augment the base file's stat information before returning. */
-       smb_fname->st.st_ex_size = get_xattr_size(handle->conn, NULL,
-                                                 smb_fname->base_name,
-                                                 xattr_name);
-       if (smb_fname->st.st_ex_size == -1) {
+       if (is_named_stream(smb_fname)) {
+               /*
+                * There can never be EA's on a symlink.
+                * Windows will never see a symlink, and
+                * in SMB_FILENAME_POSIX_PATH mode we don't
+                * allow EA's on a symlink.
+                */
+               SET_STAT_INVALID(smb_fname->st);
                errno = ENOENT;
-               result = -1;
-               goto fail;
+               return -1;
        }
-
-       smb_fname->st.st_ex_ino = stream_inode(&smb_fname->st, xattr_name);
-       smb_fname->st.st_ex_mode &= ~S_IFMT;
-        smb_fname->st.st_ex_mode |= S_IFREG;
-        smb_fname->st.st_ex_blocks =
-           smb_fname->st.st_ex_size / STAT_ST_BLOCKSIZE + 1;
-
-       result = 0;
-
- fail:
-       TALLOC_FREE(xattr_name);
-       return result;
+       return SMB_VFS_NEXT_LSTAT(handle, smb_fname);
 }
 
-static int streams_xattr_open(vfs_handle_struct *handle,
-                             struct smb_filename *smb_fname,
-                             files_struct *fsp, int flags, mode_t mode)
+static int streams_xattr_openat(struct vfs_handle_struct *handle,
+                               const struct files_struct *dirfsp,
+                               const struct smb_filename *smb_fname,
+                               files_struct *fsp,
+                               const struct vfs_open_how *how)
 {
        NTSTATUS status;
-       struct smb_filename *smb_fname_base = NULL;
-       struct stream_io *sio;
+       struct streams_xattr_config *config = NULL;
+       struct stream_io *sio = NULL;
        struct ea_struct ea;
        char *xattr_name = NULL;
-       int baseflags;
-       int hostfd = -1;
-
-       DEBUG(10, ("streams_xattr_open called for %s with flags 0x%x\n",
-                  smb_fname_str_dbg(smb_fname), flags));
-
-       if (!is_ntfs_stream_smb_fname(smb_fname)) {
-               return SMB_VFS_NEXT_OPEN(handle, smb_fname, fsp, flags, mode);
-       }
-
-       /* If the default stream is requested, just open the base file. */
-       if (is_ntfs_default_stream_smb_fname(smb_fname)) {
-               char *tmp_stream_name;
-               int ret;
+       int fakefd = -1;
+       bool set_empty_xattr = false;
+       int ret;
 
-               tmp_stream_name = smb_fname->stream_name;
-               smb_fname->stream_name = NULL;
+       SMB_VFS_HANDLE_GET_DATA(handle, config, struct streams_xattr_config,
+                               return -1);
 
-               ret = SMB_VFS_NEXT_OPEN(handle, smb_fname, fsp, flags, mode);
+       DBG_DEBUG("called for %s with flags 0x%x\n",
+                 smb_fname_str_dbg(smb_fname),
+                 how->flags);
 
-               smb_fname->stream_name = tmp_stream_name;
+       if (!is_named_stream(smb_fname)) {
+               return SMB_VFS_NEXT_OPENAT(handle,
+                                          dirfsp,
+                                          smb_fname,
+                                          fsp,
+                                          how);
+       }
 
-               return ret;
+       if (how->resolve != 0) {
+               errno = ENOSYS;
+               return -1;
        }
 
+       SMB_ASSERT(fsp_is_alternate_stream(fsp));
+       SMB_ASSERT(dirfsp == NULL);
+
        status = streams_xattr_get_name(handle, talloc_tos(),
                                        smb_fname->stream_name, &xattr_name);
        if (!NT_STATUS_IS_OK(status)) {
@@ -407,62 +352,39 @@ static int streams_xattr_open(vfs_handle_struct *handle,
                goto fail;
        }
 
-       /* Create an smb_filename with stream_name == NULL. */
-       smb_fname_base = synthetic_smb_fname(
-               talloc_tos(), smb_fname->base_name, NULL, NULL);
-       if (smb_fname_base == NULL) {
-               errno = ENOMEM;
-               goto fail;
-       }
+       status = get_ea_value_fsp(talloc_tos(),
+                                 fsp->base_fsp,
+                                 xattr_name,
+                                 &ea);
 
-       /*
-        * We use baseflags to turn off nasty side-effects when opening the
-        * underlying file.
-         */
-        baseflags = flags;
-        baseflags &= ~O_TRUNC;
-        baseflags &= ~O_EXCL;
-        baseflags &= ~O_CREAT;
-
-        hostfd = SMB_VFS_OPEN(handle->conn, smb_fname_base, fsp,
-                             baseflags, mode);
-
-       TALLOC_FREE(smb_fname_base);
-
-        /* It is legit to open a stream on a directory, but the base
-         * fd has to be read-only.
-         */
-        if ((hostfd == -1) && (errno == EISDIR)) {
-                baseflags &= ~O_ACCMODE;
-                baseflags |= O_RDONLY;
-                hostfd = SMB_VFS_OPEN(handle->conn, smb_fname, fsp, baseflags,
-                                     mode);
-        }
+       DBG_DEBUG("get_ea_value_fsp returned %s\n", nt_errstr(status));
 
-        if (hostfd == -1) {
-               goto fail;
-        }
+       if (!NT_STATUS_IS_OK(status)) {
+               if (!NT_STATUS_EQUAL(status, NT_STATUS_NOT_FOUND)) {
+                       /*
+                        * The base file is not there. This is an error even if
+                        * we got O_CREAT, the higher levels should have created
+                        * the base file for us.
+                        */
+                       DBG_DEBUG("streams_xattr_open: base file %s not around, "
+                                 "returning ENOENT\n", smb_fname->base_name);
+                       errno = ENOENT;
+                       goto fail;
+               }
 
-       status = get_ea_value(talloc_tos(), handle->conn, NULL,
-                             smb_fname->base_name, xattr_name, &ea);
+               if (!(how->flags & O_CREAT)) {
+                       errno = ENOATTR;
+                       goto fail;
+               }
 
-       DEBUG(10, ("get_ea_value returned %s\n", nt_errstr(status)));
+               set_empty_xattr = true;
+       }
 
-       if (!NT_STATUS_IS_OK(status)
-           && !NT_STATUS_EQUAL(status, NT_STATUS_NOT_FOUND)) {
-               /*
-                * The base file is not there. This is an error even if we got
-                * O_CREAT, the higher levels should have created the base
-                * file for us.
-                */
-               DEBUG(10, ("streams_xattr_open: base file %s not around, "
-                          "returning ENOENT\n", smb_fname->base_name));
-               errno = ENOENT;
-               goto fail;
+       if (how->flags & O_TRUNC) {
+               set_empty_xattr = true;
        }
 
-       if ((!NT_STATUS_IS_OK(status) && (flags & O_CREAT)) ||
-           (flags & O_TRUNC)) {
+       if (set_empty_xattr) {
                /*
                 * The attribute does not exist or needs to be truncated
                 */
@@ -475,26 +397,18 @@ static int streams_xattr_open(vfs_handle_struct *handle,
                DEBUG(10, ("creating or truncating attribute %s on file %s\n",
                           xattr_name, smb_fname->base_name));
 
-               if (fsp->base_fsp->fh->fd != -1) {
-                       if (SMB_VFS_FSETXATTR(
-                                       fsp->base_fsp, xattr_name,
-                                       &null, sizeof(null),
-                                       flags & O_EXCL ? XATTR_CREATE : 0) == -1) {
-                               goto fail;
-                       }
-               } else {
-                       if (SMB_VFS_SETXATTR(
-                                       handle->conn, smb_fname->base_name,
-                                       xattr_name, &null, sizeof(null),
-                                       flags & O_EXCL ? XATTR_CREATE : 0) == -1) {
-                               goto fail;
-                       }
+               ret = SMB_VFS_FSETXATTR(fsp->base_fsp,
+                                      xattr_name,
+                                      &null, sizeof(null),
+                                      how->flags & O_EXCL ? XATTR_CREATE : 0);
+               if (ret != 0) {
+                       goto fail;
                }
        }
 
-        sio = (struct stream_io *)VFS_ADD_FSP_EXTENSION(handle, fsp,
-                                                       struct stream_io,
-                                                       NULL);
+       fakefd = vfs_fake_fd();
+
+        sio = VFS_ADD_FSP_EXTENSION(handle, fsp, struct stream_io, NULL);
         if (sio == NULL) {
                 errno = ENOMEM;
                 goto fail;
@@ -502,59 +416,82 @@ static int streams_xattr_open(vfs_handle_struct *handle,
 
         sio->xattr_name = talloc_strdup(VFS_MEMCTX_FSP_EXTENSION(handle, fsp),
                                        xattr_name);
-        sio->base = talloc_strdup(VFS_MEMCTX_FSP_EXTENSION(handle, fsp),
-                                 smb_fname->base_name);
-       sio->fsp_name_ptr = fsp->fsp_name;
-       sio->handle = handle;
-       sio->fsp = fsp;
+       if (sio->xattr_name == NULL) {
+               errno = ENOMEM;
+               goto fail;
+       }
 
-       if ((sio->xattr_name == NULL) || (sio->base == NULL)) {
+       /*
+        * so->base needs to be a copy of fsp->fsp_name->base_name,
+        * making it identical to streams_xattr_recheck(). If the
+        * open is changing directories, fsp->fsp_name->base_name
+        * will be the full path from the share root, whilst
+        * smb_fname will be relative to the $cwd.
+        */
+        sio->base = talloc_strdup(VFS_MEMCTX_FSP_EXTENSION(handle, fsp),
+                                 fsp->fsp_name->base_name);
+       if (sio->base == NULL) {
                errno = ENOMEM;
                goto fail;
        }
 
-       return hostfd;
+       sio->fsp_name_ptr = fsp->fsp_name;
+       sio->handle = handle;
+       sio->fsp = fsp;
+
+       return fakefd;
 
  fail:
-       if (hostfd >= 0) {
-               /*
-                * BUGBUGBUG -- we would need to call fd_close_posix here, but
-                * we don't have a full fsp yet
-                */
-               fsp->fh->fd = hostfd;
-               SMB_VFS_CLOSE(fsp);
+       if (fakefd >= 0) {
+               vfs_fake_fd_close(fakefd);
+               fakefd = -1;
        }
 
        return -1;
 }
 
-static int streams_xattr_unlink(vfs_handle_struct *handle,
-                               const struct smb_filename *smb_fname)
+static int streams_xattr_close(vfs_handle_struct *handle,
+                              files_struct *fsp)
 {
-       NTSTATUS status;
-       int ret = -1;
-       char *xattr_name;
+       int ret;
+       int fd;
+
+       fd = fsp_get_pathref_fd(fsp);
 
-       if (!is_ntfs_stream_smb_fname(smb_fname)) {
-               return SMB_VFS_NEXT_UNLINK(handle, smb_fname);
+       DBG_DEBUG("streams_xattr_close called [%s] fd [%d]\n",
+                       smb_fname_str_dbg(fsp->fsp_name), fd);
+
+       if (!fsp_is_alternate_stream(fsp)) {
+               return SMB_VFS_NEXT_CLOSE(handle, fsp);
        }
 
-       /* If the default stream is requested, just open the base file. */
-       if (is_ntfs_default_stream_smb_fname(smb_fname)) {
-               struct smb_filename *smb_fname_base = NULL;
+       ret = vfs_fake_fd_close(fd);
+       fsp_set_fd(fsp, -1);
 
-               smb_fname_base = cp_smb_filename(talloc_tos(), smb_fname);
-               if (smb_fname_base == NULL) {
-                       errno = ENOMEM;
-                       return -1;
-               }
+       return ret;
+}
 
-               ret = SMB_VFS_NEXT_UNLINK(handle, smb_fname_base);
+static int streams_xattr_unlinkat(vfs_handle_struct *handle,
+                       struct files_struct *dirfsp,
+                       const struct smb_filename *smb_fname,
+                       int flags)
+{
+       NTSTATUS status;
+       int ret = -1;
+       char *xattr_name = NULL;
+       struct smb_filename *pathref = NULL;
+       struct files_struct *fsp = smb_fname->fsp;
 
-               TALLOC_FREE(smb_fname_base);
-               return ret;
+       if (!is_named_stream(smb_fname)) {
+               return SMB_VFS_NEXT_UNLINKAT(handle,
+                                       dirfsp,
+                                       smb_fname,
+                                       flags);
        }
 
+       /* A stream can never be rmdir'ed */
+       SMB_ASSERT((flags & AT_REMOVEDIR) == 0);
+
        status = streams_xattr_get_name(handle, talloc_tos(),
                                        smb_fname->stream_name, &xattr_name);
        if (!NT_STATUS_IS_OK(status)) {
@@ -562,7 +499,26 @@ static int streams_xattr_unlink(vfs_handle_struct *handle,
                goto fail;
        }
 
-       ret = SMB_VFS_REMOVEXATTR(handle->conn, smb_fname->base_name, xattr_name);
+       if (fsp == NULL) {
+               status = synthetic_pathref(talloc_tos(),
+                                       handle->conn->cwd_fsp,
+                                       smb_fname->base_name,
+                                       NULL,
+                                       NULL,
+                                       smb_fname->twrp,
+                                       smb_fname->flags,
+                                       &pathref);
+               if (!NT_STATUS_IS_OK(status)) {
+                       errno = ENOENT;
+                       goto fail;
+               }
+               fsp = pathref->fsp;
+       } else {
+               SMB_ASSERT(fsp_is_alternate_stream(smb_fname->fsp));
+               fsp = fsp->base_fsp;
+       }
+
+       ret = SMB_VFS_FREMOVEXATTR(fsp, xattr_name);
 
        if ((ret == -1) && (errno == ENOATTR)) {
                errno = ENOENT;
@@ -573,11 +529,14 @@ static int streams_xattr_unlink(vfs_handle_struct *handle,
 
  fail:
        TALLOC_FREE(xattr_name);
+       TALLOC_FREE(pathref);
        return ret;
 }
 
-static int streams_xattr_rename(vfs_handle_struct *handle,
+static int streams_xattr_renameat(vfs_handle_struct *handle,
+                               files_struct *srcfsp,
                                const struct smb_filename *smb_fname_src,
+                               files_struct *dstfsp,
                                const struct smb_filename *smb_fname_dst)
 {
        NTSTATUS status;
@@ -588,13 +547,20 @@ static int streams_xattr_rename(vfs_handle_struct *handle,
        ssize_t oret;
        ssize_t nret;
        struct ea_struct ea;
+       struct smb_filename *pathref_src = NULL;
+       struct smb_filename *pathref_dst = NULL;
+       struct smb_filename *full_src = NULL;
+       struct smb_filename *full_dst = NULL;
 
        src_is_stream = is_ntfs_stream_smb_fname(smb_fname_src);
        dst_is_stream = is_ntfs_stream_smb_fname(smb_fname_dst);
 
        if (!src_is_stream && !dst_is_stream) {
-               return SMB_VFS_NEXT_RENAME(handle, smb_fname_src,
-                                          smb_fname_dst);
+               return SMB_VFS_NEXT_RENAMEAT(handle,
+                                       srcfsp,
+                                       smb_fname_src,
+                                       dstfsp,
+                                       smb_fname_dst);
        }
 
        /* For now don't allow renames from or to the default stream. */
@@ -626,18 +592,66 @@ static int streams_xattr_rename(vfs_handle_struct *handle,
                goto fail;
        }
 
-       /* read the old stream */
-       status = get_ea_value(talloc_tos(), handle->conn, NULL,
-                             smb_fname_src->base_name, src_xattr_name, &ea);
+       full_src = full_path_from_dirfsp_atname(talloc_tos(),
+                                               srcfsp,
+                                               smb_fname_src);
+       if (full_src == NULL) {
+               errno = ENOMEM;
+               goto fail;
+       }
+       full_dst = full_path_from_dirfsp_atname(talloc_tos(),
+                                               dstfsp,
+                                               smb_fname_dst);
+       if (full_dst == NULL) {
+               errno = ENOMEM;
+               goto fail;
+       }
+
+       /* Get a pathref for full_src (base file, no stream name). */
+       status = synthetic_pathref(talloc_tos(),
+                               handle->conn->cwd_fsp,
+                               full_src->base_name,
+                               NULL,
+                               NULL,
+                               full_src->twrp,
+                               full_src->flags,
+                               &pathref_src);
        if (!NT_STATUS_IS_OK(status)) {
                errno = ENOENT;
                goto fail;
        }
 
-       /* (over)write the new stream */
-       nret = SMB_VFS_SETXATTR(handle->conn, smb_fname_src->base_name,
-                               dst_xattr_name, ea.value.data, ea.value.length,
-                               0);
+       /* Read the old stream from the base file fsp. */
+       status = get_ea_value_fsp(talloc_tos(),
+                                 pathref_src->fsp,
+                                 src_xattr_name,
+                                 &ea);
+       if (!NT_STATUS_IS_OK(status)) {
+               errno = map_errno_from_nt_status(status);
+               goto fail;
+       }
+
+       /* Get a pathref for full_dst (base file, no stream name). */
+       status = synthetic_pathref(talloc_tos(),
+                               handle->conn->cwd_fsp,
+                               full_dst->base_name,
+                               NULL,
+                               NULL,
+                               full_dst->twrp,
+                               full_dst->flags,
+                               &pathref_dst);
+       if (!NT_STATUS_IS_OK(status)) {
+               errno = ENOENT;
+               goto fail;
+       }
+
+       /* (Over)write the new stream on the base file fsp. */
+       nret = SMB_VFS_FSETXATTR(
+                       pathref_dst->fsp,
+                       dst_xattr_name,
+                       ea.value.data,
+                       ea.value.length,
+                       0);
        if (nret < 0) {
                if (errno == ENOATTR) {
                        errno = ENOENT;
@@ -645,9 +659,11 @@ static int streams_xattr_rename(vfs_handle_struct *handle,
                goto fail;
        }
 
-       /* remove the old stream */
-       oret = SMB_VFS_REMOVEXATTR(handle->conn, smb_fname_src->base_name,
-                                  src_xattr_name);
+       /*
+        * Remove the old stream from the base file fsp.
+        */
+       oret = SMB_VFS_FREMOVEXATTR(pathref_src->fsp,
+                                   src_xattr_name);
        if (oret < 0) {
                if (errno == ENOATTR) {
                        errno = ENOENT;
@@ -659,16 +675,21 @@ static int streams_xattr_rename(vfs_handle_struct *handle,
        errno = 0;
        ret = 0;
  fail:
+       TALLOC_FREE(pathref_src);
+       TALLOC_FREE(pathref_dst);
+       TALLOC_FREE(full_src);
+       TALLOC_FREE(full_dst);
        TALLOC_FREE(src_xattr_name);
        TALLOC_FREE(dst_xattr_name);
        return ret;
 }
 
-static NTSTATUS walk_xattr_streams(vfs_handle_struct *handle, files_struct *fsp,
-                                  const char *fname,
-                                  bool (*fn)(struct ea_struct *ea,
-                                             void *private_data),
-                                  void *private_data)
+static NTSTATUS walk_xattr_streams(vfs_handle_struct *handle,
+                               files_struct *fsp,
+                               const struct smb_filename *smb_fname,
+                               bool (*fn)(struct ea_struct *ea,
+                                       void *private_data),
+                               void *private_data)
 {
        NTSTATUS status;
        char **names;
@@ -678,8 +699,10 @@ static NTSTATUS walk_xattr_streams(vfs_handle_struct *handle, files_struct *fsp,
        SMB_VFS_HANDLE_GET_DATA(handle, config, struct streams_xattr_config,
                                return NT_STATUS_UNSUCCESSFUL);
 
-       status = get_ea_names_from_file(talloc_tos(), handle->conn, fsp, fname,
-                                       &names, &num_names);
+       status = get_ea_names_from_fsp(talloc_tos(),
+                               smb_fname->fsp,
+                               &names,
+                               &num_names);
        if (!NT_STATUS_IS_OK(status)) {
                return status;
        }
@@ -710,11 +733,15 @@ static NTSTATUS walk_xattr_streams(vfs_handle_struct *handle, files_struct *fsp,
                        continue;
                }
 
-               status = get_ea_value(names, handle->conn, fsp, fname,
-                                     names[i], &ea);
+               status = get_ea_value_fsp(names,
+                                         smb_fname->fsp,
+                                         names[i],
+                                         &ea);
                if (!NT_STATUS_IS_OK(status)) {
                        DEBUG(10, ("Could not get ea %s for file %s: %s\n",
-                                  names[i], fname, nt_errstr(status)));
+                               names[i],
+                               smb_fname->base_name,
+                               nt_errstr(status)));
                        continue;
                }
 
@@ -790,60 +817,26 @@ static bool collect_one_stream(struct ea_struct *ea, void *private_data)
        return true;
 }
 
-static NTSTATUS streams_xattr_streaminfo(vfs_handle_struct *handle,
+static NTSTATUS streams_xattr_fstreaminfo(vfs_handle_struct *handle,
                                         struct files_struct *fsp,
-                                        const char *fname,
                                         TALLOC_CTX *mem_ctx,
                                         unsigned int *pnum_streams,
                                         struct stream_struct **pstreams)
 {
-       SMB_STRUCT_STAT sbuf;
-       int ret;
        NTSTATUS status;
        struct streaminfo_state state;
 
-       if ((fsp != NULL) && (fsp->fh->fd != -1)) {
-               ret = SMB_VFS_FSTAT(fsp, &sbuf);
-       }
-       else {
-               struct smb_filename *smb_fname = NULL;
-               smb_fname = synthetic_smb_fname(talloc_tos(), fname, NULL,
-                                               NULL);
-               if (smb_fname == NULL) {
-                       return NT_STATUS_NO_MEMORY;
-               }
-               if (lp_posix_pathnames()) {
-                       ret = SMB_VFS_LSTAT(handle->conn, smb_fname);
-               } else {
-                       ret = SMB_VFS_STAT(handle->conn, smb_fname);
-               }
-               sbuf = smb_fname->st;
-               TALLOC_FREE(smb_fname);
-       }
-
-       if (ret == -1) {
-               return map_nt_error_from_unix(errno);
-       }
-
        state.streams = *pstreams;
        state.num_streams = *pnum_streams;
        state.mem_ctx = mem_ctx;
        state.handle = handle;
        state.status = NT_STATUS_OK;
 
-       if (S_ISLNK(sbuf.st_ex_mode)) {
-               /*
-                * Currently we do't have SMB_VFS_LLISTXATTR
-                * inside the VFS which means there's no way
-                * to cope with a symlink when lp_posix_pathnames().
-                * returns true. For now ignore links.
-                * FIXME - by adding SMB_VFS_LLISTXATTR. JRA.
-                */
-               status = NT_STATUS_OK;
-       } else {
-               status = walk_xattr_streams(handle, fsp, fname,
-                                   collect_one_stream, &state);
-       }
+       status = walk_xattr_streams(handle,
+                                   fsp,
+                                   fsp->fsp_name,
+                                   collect_one_stream,
+                                   &state);
 
        if (!NT_STATUS_IS_OK(status)) {
                TALLOC_FREE(state.streams);
@@ -858,7 +851,11 @@ static NTSTATUS streams_xattr_streaminfo(vfs_handle_struct *handle,
        *pnum_streams = state.num_streams;
        *pstreams = state.streams;
 
-       return SMB_VFS_NEXT_STREAMINFO(handle, fsp, fname, mem_ctx, pnum_streams, pstreams);
+       return SMB_VFS_NEXT_FSTREAMINFO(handle,
+                       fsp,
+                       mem_ctx,
+                       pnum_streams,
+                       pstreams);
 }
 
 static uint32_t streams_xattr_fs_capabilities(struct vfs_handle_struct *handle,
@@ -873,6 +870,12 @@ static int streams_xattr_connect(vfs_handle_struct *handle,
        struct streams_xattr_config *config;
        const char *default_prefix = SAMBA_XATTR_DOSSTREAM_PREFIX;
        const char *prefix;
+       int rc;
+
+       rc = SMB_VFS_NEXT_CONNECT(handle, service, user);
+       if (rc != 0) {
+               return rc;
+       }
 
        config = talloc_zero(handle->conn, struct streams_xattr_config);
        if (config == NULL) {
@@ -925,16 +928,39 @@ static ssize_t streams_xattr_pwrite(vfs_handle_struct *handle,
                return -1;
        }
 
-       status = get_ea_value(talloc_tos(), handle->conn, fsp->base_fsp,
-                             sio->base, sio->xattr_name, &ea);
+       if ((offset + n) >= lp_smbd_max_xattr_size(SNUM(handle->conn))) {
+               /*
+                * Requested write is beyond what can be read based on
+                * samba configuration.
+                * ReFS returns STATUS_FILESYSTEM_LIMITATION, which causes
+                * entire file to be skipped by File Explorer. VFAT returns
+                * NT_STATUS_OBJECT_NAME_COLLISION causes user to be prompted
+                * to skip writing metadata, but copy data.
+                */
+               DBG_ERR("Write to xattr [%s] on file [%s] exceeds maximum "
+                       "supported extended attribute size. "
+                       "Depending on filesystem type and operating system "
+                       "(OS) specifics, this value may be increased using "
+                       "the value of the parameter: "
+                       "smbd max xattr size = <bytes>. Consult OS and "
+                       "filesystem manpages prior to increasing this limit.\n",
+                       sio->xattr_name, sio->base);
+               errno = EOVERFLOW;
+               return -1;
+       }
+
+       status = get_ea_value_fsp(talloc_tos(),
+                                 fsp->base_fsp,
+                                 sio->xattr_name,
+                                 &ea);
        if (!NT_STATUS_IS_OK(status)) {
                return -1;
        }
 
         if ((offset + n) > ea.value.length-1) {
-               uint8 *tmp;
+               uint8_t *tmp;
 
-               tmp = talloc_realloc(talloc_tos(), ea.value.data, uint8,
+               tmp = talloc_realloc(talloc_tos(), ea.value.data, uint8_t,
                                           offset + n + 1);
 
                if (tmp == NULL) {
@@ -949,16 +975,11 @@ static ssize_t streams_xattr_pwrite(vfs_handle_struct *handle,
 
         memcpy(ea.value.data + offset, data, n);
 
-       if (fsp->base_fsp->fh->fd != -1) {
-               ret = SMB_VFS_FSETXATTR(fsp->base_fsp,
-                               sio->xattr_name,
-                               ea.value.data, ea.value.length, 0);
-       } else {
-               ret = SMB_VFS_SETXATTR(fsp->conn,
-                                      fsp->base_fsp->fsp_name->base_name,
+       ret = SMB_VFS_FSETXATTR(fsp->base_fsp,
                                sio->xattr_name,
-                               ea.value.data, ea.value.length, 0);
-       }
+                               ea.value.data,
+                               ea.value.length,
+                               0);
        TALLOC_FREE(ea.value.data);
 
        if (ret == -1) {
@@ -989,16 +1010,18 @@ static ssize_t streams_xattr_pread(vfs_handle_struct *handle,
                return -1;
        }
 
-       status = get_ea_value(talloc_tos(), handle->conn, fsp->base_fsp,
-                             sio->base, sio->xattr_name, &ea);
+       status = get_ea_value_fsp(talloc_tos(),
+                                 fsp->base_fsp,
+                                 sio->xattr_name,
+                                 &ea);
        if (!NT_STATUS_IS_OK(status)) {
                return -1;
        }
 
        length = ea.value.length-1;
 
-       DEBUG(10, ("streams_xattr_pread: get_ea_value returned %d bytes\n",
-                  (int)length));
+       DBG_DEBUG("get_ea_value_fsp returned %d bytes\n",
+                  (int)length);
 
         /* Attempt to read past EOF. */
         if (length <= offset) {
@@ -1012,12 +1035,172 @@ static ssize_t streams_xattr_pread(vfs_handle_struct *handle,
         return overlap;
 }
 
+struct streams_xattr_pread_state {
+       ssize_t nread;
+       struct vfs_aio_state vfs_aio_state;
+};
+
+static void streams_xattr_pread_done(struct tevent_req *subreq);
+
+static struct tevent_req *streams_xattr_pread_send(
+       struct vfs_handle_struct *handle,
+       TALLOC_CTX *mem_ctx,
+       struct tevent_context *ev,
+       struct files_struct *fsp,
+       void *data,
+       size_t n, off_t offset)
+{
+       struct tevent_req *req = NULL;
+       struct tevent_req *subreq = NULL;
+       struct streams_xattr_pread_state *state = NULL;
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       req = tevent_req_create(mem_ctx, &state,
+                               struct streams_xattr_pread_state);
+       if (req == NULL) {
+               return NULL;
+       }
+
+       if (sio == NULL) {
+               subreq = SMB_VFS_NEXT_PREAD_SEND(state, ev, handle, fsp,
+                                                data, n, offset);
+               if (tevent_req_nomem(req, subreq)) {
+                       return tevent_req_post(req, ev);
+               }
+               tevent_req_set_callback(subreq, streams_xattr_pread_done, req);
+               return req;
+       }
+
+       state->nread = SMB_VFS_PREAD(fsp, data, n, offset);
+       if (state->nread != n) {
+               if (state->nread != -1) {
+                       errno = EIO;
+               }
+               tevent_req_error(req, errno);
+               return tevent_req_post(req, ev);
+       }
+
+       tevent_req_done(req);
+       return tevent_req_post(req, ev);
+}
+
+static void streams_xattr_pread_done(struct tevent_req *subreq)
+{
+       struct tevent_req *req = tevent_req_callback_data(
+               subreq, struct tevent_req);
+       struct streams_xattr_pread_state *state = tevent_req_data(
+               req, struct streams_xattr_pread_state);
+
+       state->nread = SMB_VFS_PREAD_RECV(subreq, &state->vfs_aio_state);
+       TALLOC_FREE(subreq);
+
+       if (tevent_req_error(req, state->vfs_aio_state.error)) {
+               return;
+       }
+       tevent_req_done(req);
+}
+
+static ssize_t streams_xattr_pread_recv(struct tevent_req *req,
+                                       struct vfs_aio_state *vfs_aio_state)
+{
+       struct streams_xattr_pread_state *state = tevent_req_data(
+               req, struct streams_xattr_pread_state);
+
+       if (tevent_req_is_unix_error(req, &vfs_aio_state->error)) {
+               return -1;
+       }
+
+       *vfs_aio_state = state->vfs_aio_state;
+       return state->nread;
+}
+
+struct streams_xattr_pwrite_state {
+       ssize_t nwritten;
+       struct vfs_aio_state vfs_aio_state;
+};
+
+static void streams_xattr_pwrite_done(struct tevent_req *subreq);
+
+static struct tevent_req *streams_xattr_pwrite_send(
+       struct vfs_handle_struct *handle,
+       TALLOC_CTX *mem_ctx,
+       struct tevent_context *ev,
+       struct files_struct *fsp,
+       const void *data,
+       size_t n, off_t offset)
+{
+       struct tevent_req *req = NULL;
+       struct tevent_req *subreq = NULL;
+       struct streams_xattr_pwrite_state *state = NULL;
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       req = tevent_req_create(mem_ctx, &state,
+                               struct streams_xattr_pwrite_state);
+       if (req == NULL) {
+               return NULL;
+       }
+
+       if (sio == NULL) {
+               subreq = SMB_VFS_NEXT_PWRITE_SEND(state, ev, handle, fsp,
+                                                 data, n, offset);
+               if (tevent_req_nomem(req, subreq)) {
+                       return tevent_req_post(req, ev);
+               }
+               tevent_req_set_callback(subreq, streams_xattr_pwrite_done, req);
+               return req;
+       }
+
+       state->nwritten = SMB_VFS_PWRITE(fsp, data, n, offset);
+       if (state->nwritten != n) {
+               if (state->nwritten != -1) {
+                       errno = EIO;
+               }
+               tevent_req_error(req, errno);
+               return tevent_req_post(req, ev);
+       }
+
+       tevent_req_done(req);
+       return tevent_req_post(req, ev);
+}
+
+static void streams_xattr_pwrite_done(struct tevent_req *subreq)
+{
+       struct tevent_req *req = tevent_req_callback_data(
+               subreq, struct tevent_req);
+       struct streams_xattr_pwrite_state *state = tevent_req_data(
+               req, struct streams_xattr_pwrite_state);
+
+       state->nwritten = SMB_VFS_PWRITE_RECV(subreq, &state->vfs_aio_state);
+       TALLOC_FREE(subreq);
+
+       if (tevent_req_error(req, state->vfs_aio_state.error)) {
+               return;
+       }
+       tevent_req_done(req);
+}
+
+static ssize_t streams_xattr_pwrite_recv(struct tevent_req *req,
+                                        struct vfs_aio_state *vfs_aio_state)
+{
+       struct streams_xattr_pwrite_state *state = tevent_req_data(
+               req, struct streams_xattr_pwrite_state);
+
+       if (tevent_req_is_unix_error(req, &vfs_aio_state->error)) {
+               return -1;
+       }
+
+       *vfs_aio_state = state->vfs_aio_state;
+       return state->nwritten;
+}
+
 static int streams_xattr_ftruncate(struct vfs_handle_struct *handle,
                                        struct files_struct *fsp,
                                        off_t offset)
 {
        int ret;
-       uint8 *tmp;
+       uint8_t *tmp;
        struct ea_struct ea;
        NTSTATUS status;
         struct stream_io *sio =
@@ -1034,13 +1217,15 @@ static int streams_xattr_ftruncate(struct vfs_handle_struct *handle,
                return -1;
        }
 
-       status = get_ea_value(talloc_tos(), handle->conn, fsp->base_fsp,
-                             sio->base, sio->xattr_name, &ea);
+       status = get_ea_value_fsp(talloc_tos(),
+                                 fsp->base_fsp,
+                                 sio->xattr_name,
+                                 &ea);
        if (!NT_STATUS_IS_OK(status)) {
                return -1;
        }
 
-       tmp = talloc_realloc(talloc_tos(), ea.value.data, uint8,
+       tmp = talloc_realloc(talloc_tos(), ea.value.data, uint8_t,
                                   offset + 1);
 
        if (tmp == NULL) {
@@ -1059,16 +1244,11 @@ static int streams_xattr_ftruncate(struct vfs_handle_struct *handle,
        ea.value.length = offset + 1;
        ea.value.data[offset] = 0;
 
-       if (fsp->base_fsp->fh->fd != -1) {
-               ret = SMB_VFS_FSETXATTR(fsp->base_fsp,
-                               sio->xattr_name,
-                               ea.value.data, ea.value.length, 0);
-       } else {
-               ret = SMB_VFS_SETXATTR(fsp->conn,
-                                      fsp->base_fsp->fsp_name->base_name,
+       ret = SMB_VFS_FSETXATTR(fsp->base_fsp,
                                sio->xattr_name,
-                               ea.value.data, ea.value.length, 0);
-       }
+                               ea.value.data,
+                               ea.value.length,
+                               0);
 
        TALLOC_FREE(ea.value.data);
 
@@ -1081,7 +1261,7 @@ static int streams_xattr_ftruncate(struct vfs_handle_struct *handle,
 
 static int streams_xattr_fallocate(struct vfs_handle_struct *handle,
                                        struct files_struct *fsp,
-                                       enum vfs_fallocate_mode mode,
+                                       uint32_t mode,
                                        off_t offset,
                                        off_t len)
 {
@@ -1097,32 +1277,337 @@ static int streams_xattr_fallocate(struct vfs_handle_struct *handle,
        }
 
        if (!streams_xattr_recheck(sio)) {
-               return errno;
+               return -1;
        }
 
        /* Let the pwrite code path handle it. */
-       return ENOSYS;
+       errno = ENOSYS;
+       return -1;
+}
+
+static int streams_xattr_fchown(vfs_handle_struct *handle, files_struct *fsp,
+                               uid_t uid, gid_t gid)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_FCHOWN(handle, fsp, uid, gid);
+       }
+
+       return 0;
 }
 
+static int streams_xattr_fchmod(vfs_handle_struct *handle,
+                               files_struct *fsp,
+                               mode_t mode)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_FCHMOD(handle, fsp, mode);
+       }
+
+       return 0;
+}
+
+static ssize_t streams_xattr_fgetxattr(struct vfs_handle_struct *handle,
+                                      struct files_struct *fsp,
+                                      const char *name,
+                                      void *value,
+                                      size_t size)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_FGETXATTR(handle, fsp, name, value, size);
+       }
+
+       errno = ENOTSUP;
+       return -1;
+}
+
+static ssize_t streams_xattr_flistxattr(struct vfs_handle_struct *handle,
+                                       struct files_struct *fsp,
+                                       char *list,
+                                       size_t size)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_FLISTXATTR(handle, fsp, list, size);
+       }
+
+       errno = ENOTSUP;
+       return -1;
+}
+
+static int streams_xattr_fremovexattr(struct vfs_handle_struct *handle,
+                                     struct files_struct *fsp,
+                                     const char *name)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_FREMOVEXATTR(handle, fsp, name);
+       }
+
+       errno = ENOTSUP;
+       return -1;
+}
+
+static int streams_xattr_fsetxattr(struct vfs_handle_struct *handle,
+                                  struct files_struct *fsp,
+                                  const char *name,
+                                  const void *value,
+                                  size_t size,
+                                  int flags)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_FSETXATTR(handle, fsp, name, value,
+                                             size, flags);
+       }
+
+       errno = ENOTSUP;
+       return -1;
+}
+
+struct streams_xattr_fsync_state {
+       int ret;
+       struct vfs_aio_state vfs_aio_state;
+};
+
+static void streams_xattr_fsync_done(struct tevent_req *subreq);
+
+static struct tevent_req *streams_xattr_fsync_send(
+       struct vfs_handle_struct *handle,
+       TALLOC_CTX *mem_ctx,
+       struct tevent_context *ev,
+       struct files_struct *fsp)
+{
+       struct tevent_req *req = NULL;
+       struct tevent_req *subreq = NULL;
+       struct streams_xattr_fsync_state *state = NULL;
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       req = tevent_req_create(mem_ctx, &state,
+                               struct streams_xattr_fsync_state);
+       if (req == NULL) {
+               return NULL;
+       }
+
+       if (sio == NULL) {
+               subreq = SMB_VFS_NEXT_FSYNC_SEND(state, ev, handle, fsp);
+               if (tevent_req_nomem(req, subreq)) {
+                       return tevent_req_post(req, ev);
+               }
+               tevent_req_set_callback(subreq, streams_xattr_fsync_done, req);
+               return req;
+       }
+
+       /*
+        * There's no pathname based sync variant and we don't have access to
+        * the basefile handle, so we can't do anything here.
+        */
+
+       tevent_req_done(req);
+       return tevent_req_post(req, ev);
+}
+
+static void streams_xattr_fsync_done(struct tevent_req *subreq)
+{
+       struct tevent_req *req = tevent_req_callback_data(
+               subreq, struct tevent_req);
+       struct streams_xattr_fsync_state *state = tevent_req_data(
+               req, struct streams_xattr_fsync_state);
+
+       state->ret = SMB_VFS_FSYNC_RECV(subreq, &state->vfs_aio_state);
+       TALLOC_FREE(subreq);
+       if (state->ret != 0) {
+               tevent_req_error(req, errno);
+               return;
+       }
+
+       tevent_req_done(req);
+}
+
+static int streams_xattr_fsync_recv(struct tevent_req *req,
+                                   struct vfs_aio_state *vfs_aio_state)
+{
+       struct streams_xattr_fsync_state *state = tevent_req_data(
+               req, struct streams_xattr_fsync_state);
+
+       if (tevent_req_is_unix_error(req, &vfs_aio_state->error)) {
+               return -1;
+       }
+
+       *vfs_aio_state = state->vfs_aio_state;
+       return state->ret;
+}
+
+static bool streams_xattr_lock(vfs_handle_struct *handle,
+                              files_struct *fsp,
+                              int op,
+                              off_t offset,
+                              off_t count,
+                              int type)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_LOCK(handle, fsp, op, offset, count, type);
+       }
+
+       return true;
+}
+
+static bool streams_xattr_getlock(vfs_handle_struct *handle,
+                                 files_struct *fsp,
+                                 off_t *poffset,
+                                 off_t *pcount,
+                                 int *ptype,
+                                 pid_t *ppid)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_GETLOCK(handle, fsp, poffset,
+                                           pcount, ptype, ppid);
+       }
+
+       errno = ENOTSUP;
+       return false;
+}
+
+static int streams_xattr_filesystem_sharemode(vfs_handle_struct *handle,
+                                             files_struct *fsp,
+                                             uint32_t share_access,
+                                             uint32_t access_mask)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_FILESYSTEM_SHAREMODE(handle,
+                                                        fsp,
+                                                        share_access,
+                                                        access_mask);
+       }
+
+       return 0;
+}
+
+static int streams_xattr_linux_setlease(vfs_handle_struct *handle,
+                                       files_struct *fsp,
+                                       int leasetype)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_LINUX_SETLEASE(handle, fsp, leasetype);
+       }
+
+       return 0;
+}
+
+static bool streams_xattr_strict_lock_check(struct vfs_handle_struct *handle,
+                                           files_struct *fsp,
+                                           struct lock_struct *plock)
+{
+       struct stream_io *sio =
+               (struct stream_io *)VFS_FETCH_FSP_EXTENSION(handle, fsp);
+
+       if (sio == NULL) {
+               return SMB_VFS_NEXT_STRICT_LOCK_CHECK(handle, fsp, plock);
+       }
+
+       return true;
+}
+
+static int streams_xattr_fcntl(vfs_handle_struct *handle,
+                              files_struct *fsp,
+                              int cmd,
+                              va_list cmd_arg)
+{
+       va_list dup_cmd_arg;
+       void *arg;
+       int ret;
+
+       if (fsp_is_alternate_stream(fsp)) {
+               switch (cmd) {
+               case F_GETFL:
+               case F_SETFL:
+                       break;
+               default:
+                       DBG_ERR("Unsupported fcntl() cmd [%d] on [%s]\n",
+                               cmd, fsp_str_dbg(fsp));
+                       errno = EINVAL;
+                       return -1;
+               }
+       }
+
+       va_copy(dup_cmd_arg, cmd_arg);
+       arg = va_arg(dup_cmd_arg, void *);
+
+       ret = SMB_VFS_NEXT_FCNTL(handle, fsp, cmd, arg);
+
+       va_end(dup_cmd_arg);
+
+       return ret;
+}
 
 static struct vfs_fn_pointers vfs_streams_xattr_fns = {
        .fs_capabilities_fn = streams_xattr_fs_capabilities,
        .connect_fn = streams_xattr_connect,
-       .open_fn = streams_xattr_open,
+       .openat_fn = streams_xattr_openat,
+       .close_fn = streams_xattr_close,
        .stat_fn = streams_xattr_stat,
        .fstat_fn = streams_xattr_fstat,
        .lstat_fn = streams_xattr_lstat,
        .pread_fn = streams_xattr_pread,
        .pwrite_fn = streams_xattr_pwrite,
-       .unlink_fn = streams_xattr_unlink,
-       .rename_fn = streams_xattr_rename,
+       .pread_send_fn = streams_xattr_pread_send,
+       .pread_recv_fn = streams_xattr_pread_recv,
+       .pwrite_send_fn = streams_xattr_pwrite_send,
+       .pwrite_recv_fn = streams_xattr_pwrite_recv,
+       .unlinkat_fn = streams_xattr_unlinkat,
+       .renameat_fn = streams_xattr_renameat,
        .ftruncate_fn = streams_xattr_ftruncate,
        .fallocate_fn = streams_xattr_fallocate,
-       .streaminfo_fn = streams_xattr_streaminfo,
+       .fstreaminfo_fn = streams_xattr_fstreaminfo,
+
+       .fsync_send_fn = streams_xattr_fsync_send,
+       .fsync_recv_fn = streams_xattr_fsync_recv,
+
+       .lock_fn = streams_xattr_lock,
+       .getlock_fn = streams_xattr_getlock,
+       .filesystem_sharemode_fn = streams_xattr_filesystem_sharemode,
+       .linux_setlease_fn = streams_xattr_linux_setlease,
+       .strict_lock_check_fn = streams_xattr_strict_lock_check,
+       .fcntl_fn = streams_xattr_fcntl,
+
+       .fchown_fn = streams_xattr_fchown,
+       .fchmod_fn = streams_xattr_fchmod,
+
+       .fgetxattr_fn = streams_xattr_fgetxattr,
+       .flistxattr_fn = streams_xattr_flistxattr,
+       .fremovexattr_fn = streams_xattr_fremovexattr,
+       .fsetxattr_fn = streams_xattr_fsetxattr,
 };
 
-NTSTATUS vfs_streams_xattr_init(void);
-NTSTATUS vfs_streams_xattr_init(void)
+static_decl_vfs;
+NTSTATUS vfs_streams_xattr_init(TALLOC_CTX *ctx)
 {
        return smb_register_vfs(SMB_VFS_INTERFACE_VERSION, "streams_xattr",
                                &vfs_streams_xattr_fns);