X-Git-Url: https://wimlib.net/git/?p=wimlib;a=blobdiff_plain;f=src%2Fmount_image.c;h=d9688765514f81baff478c2c39b15b877b2a25d6;hp=df8d523bf64aa38732689d78c5e0193d6f6f39dc;hb=7acdc3aecd7ce546727ae8239e0d942c2a00db43;hpb=01919fdd57105c0ab91316b79c10e3992e686436

diff --git a/src/mount_image.c b/src/mount_image.c
index df8d523b..d9688765 100644
--- a/src/mount_image.c
+++ b/src/mount_image.c
@@ -8,7 +8,7 @@
  */
 
 /*
- * Copyright (C) 2012 Eric Biggers
+ * Copyright (C) 2012, 2013 Eric Biggers
  *
  * This file is part of wimlib, a library for working with WIM files.
  *
@@ -30,24 +30,28 @@
 
 #ifdef WITH_FUSE
 
-#include "sha1.h"
-#include "lookup_table.h"
-#include "xml.h"
 #include "buffer_io.h"
+#include "lookup_table.h"
+#include "sha1.h"
 #include "timestamp.h"
+#include "xml.h"
+
+#include <errno.h>
+#include <ftw.h>
 #include <limits.h>
+#include <mqueue.h>
+#include <signal.h>
 #include <stdlib.h>
-#include <unistd.h>
-#include <sys/wait.h>
-#define FUSE_USE_VERSION 26
-#include <errno.h>
 #include <string.h>
+#include <sys/stat.h>
 #include <sys/time.h>
-#include <fuse.h>
-#include <ftw.h>
-#include <mqueue.h>
+#include <sys/types.h>
+#include <sys/wait.h>
+#include <unistd.h>
 #include <utime.h>
-#include <signal.h>
+
+#define FUSE_USE_VERSION 26
+#include <fuse.h>
 
 #ifdef ENABLE_XATTR
 #include <attr/xattr.h>
@@ -106,6 +110,12 @@ struct wimfs_context {
 	char *daemon_to_unmount_mq_name;
 	mqd_t unmount_to_daemon_mq;
 	mqd_t daemon_to_unmount_mq;
+
+	uid_t default_uid;
+	gid_t default_gid;
+
+	int status;
+	bool have_status;
 };
 
 static void init_wimfs_context(struct wimfs_context *ctx)
@@ -116,9 +126,11 @@ static void init_wimfs_context(struct wimfs_context *ctx)
 	INIT_LIST_HEAD(&ctx->staging_list);
 }
 
+#define WIMFS_CTX(fuse_ctx) ((struct wimfs_context*)(fuse_ctx)->private_data)
+
 static inline struct wimfs_context *wimfs_get_context()
 {
-	return (struct wimfs_context*)fuse_get_context()->private_data;
+	return WIMFS_CTX(fuse_get_context());
 }
 
 static inline WIMStruct *wimfs_get_WIMStruct()
@@ -165,7 +177,8 @@ static int alloc_wimfs_fd(struct wim_inode *inode,
 
 	pthread_mutex_lock(&inode->i_mutex);
 
-	DEBUG("Allocating fd for stream ID %u from inode %lx (open = %u, allocated = %u)",
+	DEBUG("Allocating fd for stream ID %u from inode %#"PRIx64" "
+	      "(open = %u, allocated = %u)",
 	      stream_id, inode->i_ino, inode->i_num_opened_fds,
 	      inode->i_num_allocated_fds);
 
@@ -265,7 +278,7 @@ static int lte_put_fd(struct wim_lookup_table_entry *lte, struct wimfs_fd *fd)
 static int close_wimfs_fd(struct wimfs_fd *fd)
 {
 	int ret;
-	DEBUG("Closing fd (inode = %lu, opened = %u, allocated = %u)",
+	DEBUG("Closing fd (ino = %#"PRIx64", opened = %u, allocated = %u)",
 	      fd->f_inode->i_ino, fd->f_inode->i_num_opened_fds,
 	      fd->f_inode->i_num_allocated_fds);
 	ret = lte_put_fd(fd->f_lte, fd);
@@ -279,20 +292,18 @@ static int close_wimfs_fd(struct wimfs_fd *fd)
 /*
  * Add a new dentry with a new inode to a WIM image.
  *
- * @ctx:	 Context for the mounted WIM image.
- * @path:	 Path to create the dentry at.
- * @dentry_ret:  Return the pointer to the dentry if successful.
- *
  * Returns 0 on success, or negative error number on failure.
  */
-static int create_dentry(struct wimfs_context *ctx, const char *path,
+static int create_dentry(struct fuse_context *fuse_ctx,const char *path,
+			 mode_t mode, int attributes,
 			 struct wim_dentry **dentry_ret)
 {
 	struct wim_dentry *parent;
 	struct wim_dentry *new;
 	const char *basename;
+	struct wimfs_context *wimfs_ctx = WIMFS_CTX(fuse_ctx);
 
-	parent = get_parent_dentry(ctx->wim, path);
+	parent = get_parent_dentry(wimfs_ctx->wim, path);
 	if (!parent)
 		return -errno;
 
@@ -308,10 +319,25 @@ static int create_dentry(struct wimfs_context *ctx, const char *path,
 		return -errno;
 
 	new->d_inode->i_resolved = 1;
-	new->d_inode->i_ino = ctx->next_ino++;
+	new->d_inode->i_ino = wimfs_ctx->next_ino++;
+	new->d_inode->i_attributes = attributes;
+
+	if (wimfs_ctx->mount_flags & WIMLIB_MOUNT_FLAG_UNIX_DATA) {
+		if (inode_set_unix_data(new->d_inode,
+					fuse_ctx->uid,
+					fuse_ctx->gid,
+					mode & ~fuse_ctx->umask,
+					wimfs_ctx->wim->lookup_table,
+					UNIX_DATA_ALL | UNIX_DATA_CREATE))
+		{
+			free_dentry(new);
+			return -ENOMEM;
+		}
+	}
 	dentry_add_child(parent, new);
-	hlist_add_head(&new->d_inode->i_hlist, ctx->image_inode_list);
-	*dentry_ret = new;
+	hlist_add_head(&new->d_inode->i_hlist, wimfs_ctx->image_inode_list);
+	if (dentry_ret)
+		*dentry_ret = new;
 	return 0;
 }
 
@@ -343,6 +369,16 @@ static void remove_dentry(struct wim_dentry *dentry,
 	put_dentry(dentry);
 }
 
+static mode_t inode_default_unix_mode(const struct wim_inode *inode)
+{
+	if (inode_is_symlink(inode))
+		return S_IFLNK | 0777;
+	else if (inode_is_directory(inode))
+		return S_IFDIR | 0777;
+	else
+		return S_IFREG | 0777;
+}
+
 /* Transfers file attributes from a struct wim_inode to a `stat' buffer.
  *
  * The lookup table entry tells us which stream in the inode we are statting.
@@ -352,18 +388,22 @@ static int inode_to_stbuf(const struct wim_inode *inode,
 			  const struct wim_lookup_table_entry *lte,
 			  struct stat *stbuf)
 {
-	if (inode_is_symlink(inode))
-		stbuf->st_mode = S_IFLNK | 0777;
-	else if (inode_is_directory(inode))
-		stbuf->st_mode = S_IFDIR | 0755;
-	else
-		stbuf->st_mode = S_IFREG | 0755;
-
-	stbuf->st_ino   = (ino_t)inode->i_ino;
+	const struct wimfs_context *ctx = wimfs_get_context();
+
+	memset(stbuf, 0, sizeof(struct stat));
+	stbuf->st_mode = inode_default_unix_mode(inode);
+	stbuf->st_uid = ctx->default_uid;
+	stbuf->st_gid = ctx->default_gid;
+	if (ctx->mount_flags & WIMLIB_MOUNT_FLAG_UNIX_DATA) {
+		struct wimlib_unix_data unix_data;
+		if (inode_get_unix_data(inode, &unix_data, NULL) == 0) {
+			stbuf->st_uid = unix_data.uid;
+			stbuf->st_gid = unix_data.gid;
+			stbuf->st_mode = unix_data.mode;
+		}
+	}
+	stbuf->st_ino = (ino_t)inode->i_ino;
 	stbuf->st_nlink = inode->i_nlink;
-	stbuf->st_uid   = getuid();
-	stbuf->st_gid   = getgid();
-
 	if (lte) {
 		if (lte->resource_location == RESOURCE_IN_STAGING_FILE) {
 			struct stat native_stat;
@@ -380,13 +420,26 @@ static int inode_to_stbuf(const struct wim_inode *inode,
 		stbuf->st_size = 0;
 	}
 
-	stbuf->st_atime   = wim_timestamp_to_unix(inode->i_last_access_time);
-	stbuf->st_mtime   = wim_timestamp_to_unix(inode->i_last_write_time);
-	stbuf->st_ctime   = wim_timestamp_to_unix(inode->i_creation_time);
-	stbuf->st_blocks  = (stbuf->st_size + 511) / 512;
+#ifdef HAVE_STAT_NANOSECOND_PRECISION
+	stbuf->st_atim = wim_timestamp_to_timespec(inode->i_last_access_time);
+	stbuf->st_mtim = wim_timestamp_to_timespec(inode->i_last_write_time);
+	stbuf->st_ctim = stbuf->st_mtim;
+#else
+	stbuf->st_atime = wim_timestamp_to_unix(inode->i_last_access_time);
+	stbuf->st_mtime = wim_timestamp_to_unix(inode->i_last_write_time);
+	stbuf->st_ctime = stbuf->st_mtime;
+#endif
+	stbuf->st_blocks = (stbuf->st_size + 511) / 512;
 	return 0;
 }
 
+static void touch_inode(struct wim_inode *inode)
+{
+	u64 now = get_wim_timestamp();
+	inode->i_last_access_time = now;
+	inode->i_last_write_time = now;
+}
+
 /* Creates a new staging file and returns its file descriptor opened for
  * writing.
  *
@@ -757,7 +810,7 @@ static int update_lte_of_staging_file(struct wim_lookup_table_entry *lte,
 			inode_update_lte_ptr(lte->lte_inode, lte, NULL);
 			free_lookup_table_entry(lte);
 		} else {
-			wimlib_assert(&lte->file_on_disk == &lte->staging_file_name);
+			BUILD_BUG_ON(&lte->file_on_disk != &lte->staging_file_name);
 			lte->resource_entry.original_size = stbuf.st_size;
 			lte->resource_entry.size = stbuf.st_size;
 			lte->resource_location = RESOURCE_IN_FILE_ON_DISK;
@@ -801,7 +854,7 @@ static int rebuild_wim(struct wimfs_context *ctx, int write_flags,
 	}
 
 	DEBUG("Calculating SHA1 checksums for all new staging files.");
-	list_for_each_entry(lte, &ctx->staging_list, staging_list) {
+	list_for_each_entry_safe(lte, tmp, &ctx->staging_list, staging_list) {
 		ret = update_lte_of_staging_file(lte, w->lookup_table);
 		if (ret != 0)
 			return ret;
@@ -847,12 +900,9 @@ static int set_message_queue_names(struct wimfs_context *ctx,
 			return WIMLIB_ERR_NOTDIR;
 	}
 
-	p = dir_path;
-	while (*p) {
+	for (p = dir_path; *p; p++)
 		if (*p == '/')
 			*p = 0xff;
-		p++;
-	}
 
 	ctx->unmount_to_daemon_mq_name = strcat_dup(u2d_prefix, dir_path,
 						    NAME_MAX);
@@ -898,31 +948,44 @@ static int open_message_queues(struct wimfs_context *ctx, bool daemon)
 {
 	int unmount_to_daemon_mq_flags = O_WRONLY | O_CREAT;
 	int daemon_to_unmount_mq_flags = O_RDONLY | O_CREAT;
+	mode_t mode;
+	mode_t orig_umask;
+	int ret;
 
-	if (daemon)
+	if (daemon) {
 		swap(unmount_to_daemon_mq_flags, daemon_to_unmount_mq_flags);
+		mode = 0600;
+	} else {
+		mode = 0666;
+	}
 
+	orig_umask = umask(0000);
 	DEBUG("Opening message queue \"%s\"", ctx->unmount_to_daemon_mq_name);
 	ctx->unmount_to_daemon_mq = mq_open(ctx->unmount_to_daemon_mq_name,
-					    unmount_to_daemon_mq_flags, 0700, NULL);
+					    unmount_to_daemon_mq_flags, mode, NULL);
 
 	if (ctx->unmount_to_daemon_mq == (mqd_t)-1) {
 		ERROR_WITH_ERRNO("mq_open()");
-		return WIMLIB_ERR_MQUEUE;
+		ret = WIMLIB_ERR_MQUEUE;
+		goto out;
 	}
 
 	DEBUG("Opening message queue \"%s\"", ctx->daemon_to_unmount_mq_name);
 	ctx->daemon_to_unmount_mq = mq_open(ctx->daemon_to_unmount_mq_name,
-					    daemon_to_unmount_mq_flags, 0700, NULL);
+					    daemon_to_unmount_mq_flags, mode, NULL);
 
 	if (ctx->daemon_to_unmount_mq == (mqd_t)-1) {
 		ERROR_WITH_ERRNO("mq_open()");
 		mq_close(ctx->unmount_to_daemon_mq);
 		mq_unlink(ctx->unmount_to_daemon_mq_name);
 		ctx->unmount_to_daemon_mq = (mqd_t)-1;
-		return WIMLIB_ERR_MQUEUE;
+		ret = WIMLIB_ERR_MQUEUE;
+		goto out;
 	}
-	return 0;
+	ret = 0;
+out:
+	umask(orig_umask);
+	return ret;
 }
 
 /* Try to determine the maximum message size of a message queue.  The return
@@ -1193,7 +1256,8 @@ out:
 				status = ret;
 		}
 	}
-	send_unmount_finished_msg(wimfs_ctx->daemon_to_unmount_mq, status);
+	wimfs_ctx->status = status;
+	wimfs_ctx->have_status = true;
 	return MSG_BREAK_LOOP;
 }
 
@@ -1460,31 +1524,52 @@ static int execute_fusermount(const char *dir)
 	return 0;
 }
 
+#if 0
 static int wimfs_access(const char *path, int mask)
 {
-	/* Permissions not implemented */
-	return 0;
+	return -ENOSYS;
 }
+#endif
 
 static int wimfs_chmod(const char *path, mode_t mask)
 {
 	struct wim_dentry *dentry;
 	struct wimfs_context *ctx = wimfs_get_context();
-	struct wim_inode *inode;
-	struct stat stbuf;
 	int ret;
 
-	ret = lookup_resource(ctx->wim, path,
-			      get_lookup_flags(ctx) | LOOKUP_FLAG_DIRECTORY_OK,
+	if (!(ctx->mount_flags & WIMLIB_MOUNT_FLAG_UNIX_DATA))
+		return -EPERM;
+
+	ret = lookup_resource(ctx->wim, path, LOOKUP_FLAG_DIRECTORY_OK,
 			      &dentry, NULL, NULL);
-	if (ret != 0)
+	if (ret)
 		return ret;
-	inode = dentry->d_inode;
-	inode_to_stbuf(inode, NULL, &stbuf);
-	if (mask == stbuf.st_mode)
-		return 0;
-	else
+
+	ret = inode_set_unix_data(dentry->d_inode, ctx->default_uid,
+				  ctx->default_gid, mask,
+				  ctx->wim->lookup_table, UNIX_DATA_MODE);
+	return ret ? -ENOMEM : 0;
+}
+
+static int wimfs_chown(const char *path, uid_t uid, gid_t gid)
+{
+	struct wim_dentry *dentry;
+	struct wimfs_context *ctx = wimfs_get_context();
+	int ret;
+
+	if (!(ctx->mount_flags & WIMLIB_MOUNT_FLAG_UNIX_DATA))
 		return -EPERM;
+
+	ret = lookup_resource(ctx->wim, path, LOOKUP_FLAG_DIRECTORY_OK,
+			      &dentry, NULL, NULL);
+	if (ret)
+		return ret;
+
+	ret = inode_set_unix_data(dentry->d_inode, uid, gid,
+				  inode_default_unix_mode(dentry->d_inode),
+				  ctx->wim->lookup_table,
+				  UNIX_DATA_UID | UNIX_DATA_GID);
+	return ret ? -ENOMEM : 0;
 }
 
 /* Called when the filesystem is unmounted. */
@@ -1501,7 +1586,6 @@ static void wimfs_destroy(void *p)
 		message_loop(wimfs_ctx->unmount_to_daemon_mq,
 			     &daemon_msg_handler_callbacks,
 			     &handler_ctx.hdr);
-		close_message_queues(wimfs_ctx);
 	}
 }
 
@@ -1560,7 +1644,7 @@ static int wimfs_getxattr(const char *path, const char *name, char *value,
 {
 	int ret;
 	struct wim_inode *inode;
-	struct ads_entry *ads_entry;
+	struct wim_ads_entry *ads_entry;
 	size_t res_size;
 	struct wim_lookup_table_entry *lte;
 	struct wimfs_context *ctx = wimfs_get_context();
@@ -1612,15 +1696,13 @@ static int wimfs_link(const char *to, const char *from)
 	if (!inode)
 		return -errno;
 
-	if (inode->i_attributes & FILE_ATTRIBUTE_REPARSE_POINT)
-		return -EEXIST;
-
-	if (inode->i_attributes & FILE_ATTRIBUTE_DIRECTORY)
+	if (inode->i_attributes & (FILE_ATTRIBUTE_DIRECTORY |
+				   FILE_ATTRIBUTE_REPARSE_POINT))
 		return -EPERM;
 
 	from_dentry_parent = get_parent_dentry(w, from);
 	if (!from_dentry_parent)
-		return -ENOENT;
+		return -errno;
 	if (!dentry_is_directory(from_dentry_parent))
 		return -ENOTDIR;
 
@@ -1683,36 +1765,34 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
 #endif
 
 
-/* Create a directory in the WIM.
- * @mode is currently ignored.  */
+/* Create a directory in the WIM image. */
 static int wimfs_mkdir(const char *path, mode_t mode)
 {
-	struct wim_dentry *dentry;
-	int ret;
-
-	ret = create_dentry(wimfs_get_context(), path, &dentry);
-	if (ret == 0)
-		dentry->d_inode->i_attributes = FILE_ATTRIBUTE_DIRECTORY;
-	return ret;
+	return create_dentry(fuse_get_context(), path, mode | S_IFDIR,
+			     FILE_ATTRIBUTE_DIRECTORY, NULL);
 }
 
-/* Create a regular file in the WIM.
- * @mode is currently ignored.  */
+/* Create a regular file or alternate data stream in the WIM image. */
 static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
 {
 	const char *stream_name;
-	struct wimfs_context *ctx = wimfs_get_context();
-	if ((ctx->mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_WINDOWS)
+	struct fuse_context *fuse_ctx = fuse_get_context();
+	struct wimfs_context *wimfs_ctx = WIMFS_CTX(fuse_ctx);
+
+	if (!S_ISREG(mode))
+		return -EPERM;
+
+	if ((wimfs_ctx->mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_WINDOWS)
 	     && (stream_name = path_stream_name(path))) {
 		/* Make an alternate data stream */
-		struct ads_entry *new_entry;
+		struct wim_ads_entry *new_entry;
 		struct wim_inode *inode;
 
 		char *p = (char*)stream_name - 1;
 		wimlib_assert(*p == ':');
 		*p = '\0';
 
-		inode = wim_pathname_to_inode(ctx->wim, path);
+		inode = wim_pathname_to_inode(wimfs_ctx->wim, path);
 		if (!inode)
 			return -errno;
 		if (inode->i_attributes &
@@ -1725,14 +1805,9 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
 			return -ENOMEM;
 		return 0;
 	} else {
-		struct wim_dentry *dentry;
-		int ret;
-
 		/* Make a normal file (not an alternate data stream) */
-		ret = create_dentry(ctx, path, &dentry);
-		if (ret == 0)
-			dentry->d_inode->i_attributes = FILE_ATTRIBUTE_NORMAL;
-		return ret;
+		return create_dentry(fuse_ctx, path, mode | S_IFREG,
+				     FILE_ATTRIBUTE_NORMAL, NULL);
 	}
 }
 
@@ -1778,7 +1853,7 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
 	}
 
 	ret = alloc_wimfs_fd(inode, stream_id, lte, &fd,
-			      wimfs_ctx_readonly(ctx));
+			     wimfs_ctx_readonly(ctx));
 	if (ret != 0)
 		return ret;
 
@@ -1835,7 +1910,7 @@ static int wimfs_read(const char *path, char *buf, size_t size,
 		wimlib_assert(fd->f_lte->staging_file_name);
 		wimlib_assert(fd->staging_fd != -1);
 
-		DEBUG("Seek to offset %zu", offset);
+		DEBUG("Seek to offset %"PRIu64, offset);
 
 		if (lseek(fd->staging_fd, offset, SEEK_SET) == -1)
 			return -errno;
@@ -1931,7 +2006,7 @@ static int wimfs_releasedir(const char *path, struct fuse_file_info *fi)
 static int wimfs_removexattr(const char *path, const char *name)
 {
 	struct wim_inode *inode;
-	struct ads_entry *ads_entry;
+	struct wim_ads_entry *ads_entry;
 	u16 ads_idx;
 	struct wimfs_context *ctx = wimfs_get_context();
 
@@ -2001,7 +2076,7 @@ static int wimfs_rename(const char *from, const char *to)
 			return -ENOTDIR;
 	}
 
-	ret = set_dentry_name(src, to);
+	ret = set_dentry_name(src, path_basename(to));
 	if (ret != 0)
 		return -ENOMEM;
 	if (dst)
@@ -2014,7 +2089,7 @@ static int wimfs_rename(const char *from, const char *to)
 /* Remove a directory */
 static int wimfs_rmdir(const char *path)
 {
-	struct wim_dentry *parent, *dentry;
+	struct wim_dentry *dentry;
 	WIMStruct *w = wimfs_get_WIMStruct();
 
 	dentry = get_dentry(w, path);
@@ -2036,14 +2111,11 @@ static int wimfs_rmdir(const char *path)
 static int wimfs_setxattr(const char *path, const char *name,
 			  const char *value, size_t size, int flags)
 {
-	struct ads_entry *existing_ads_entry;
-	struct ads_entry *new_ads_entry;
-	struct wim_lookup_table_entry *existing_lte;
-	struct wim_lookup_table_entry *lte;
+	struct wim_ads_entry *existing_ads_entry;
 	struct wim_inode *inode;
-	u8 value_hash[SHA1_HASH_SIZE];
 	u16 ads_idx;
 	struct wimfs_context *ctx = wimfs_get_context();
+	int ret;
 
 	if (!(ctx->mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_XATTR))
 		return -ENOTSUP;
@@ -2060,61 +2132,38 @@ static int wimfs_setxattr(const char *path, const char *name,
 	if (existing_ads_entry) {
 		if (flags & XATTR_CREATE)
 			return -EEXIST;
-		inode_remove_ads(inode, ads_idx, ctx->wim->lookup_table);
 	} else {
 		if (flags & XATTR_REPLACE)
 			return -ENOATTR;
 	}
-	new_ads_entry = inode_add_ads(inode, name);
-	if (!new_ads_entry)
-		return -ENOMEM;
-
-	sha1_buffer((const u8*)value, size, value_hash);
-
-	existing_lte = __lookup_resource(ctx->wim->lookup_table, value_hash);
 
-	if (existing_lte) {
-		lte = existing_lte;
-		lte->refcnt++;
+	ret = inode_add_ads_with_data(inode, name, (const u8*)value,
+				      size, ctx->wim->lookup_table);
+	if (ret == 0) {
+		if (existing_ads_entry)
+			inode_remove_ads(inode, ads_idx, ctx->wim->lookup_table);
 	} else {
-		u8 *value_copy;
-		lte = new_lookup_table_entry();
-		if (!lte)
-			return -ENOMEM;
-		value_copy = MALLOC(size);
-		if (!value_copy) {
-			FREE(lte);
-			return -ENOMEM;
-		}
-		memcpy(value_copy, value, size);
-		lte->resource_location            = RESOURCE_IN_ATTACHED_BUFFER;
-		lte->attached_buffer              = value_copy;
-		lte->resource_entry.original_size = size;
-		lte->resource_entry.size          = size;
-		lte->resource_entry.flags         = 0;
-		copy_hash(lte->hash, value_hash);
-		lookup_table_insert(ctx->wim->lookup_table, lte);
-	}
-	new_ads_entry->lte = lte;
-	return 0;
+		ret = -ENOMEM;
+	}
+	return ret;
 }
 #endif
 
 static int wimfs_symlink(const char *to, const char *from)
 {
-	struct wimfs_context *ctx = wimfs_get_context();
+	struct fuse_context *fuse_ctx = fuse_get_context();
+	struct wimfs_context *wimfs_ctx = WIMFS_CTX(fuse_ctx);
 	struct wim_dentry *dentry;
 	int ret;
 
-	ret = create_dentry(ctx, from, &dentry);
+	ret = create_dentry(fuse_ctx, from, S_IFLNK | 0777,
+			    FILE_ATTRIBUTE_REPARSE_POINT, &dentry);
 	if (ret == 0) {
-		dentry->d_inode->i_attributes = FILE_ATTRIBUTE_REPARSE_POINT;
 		dentry->d_inode->i_reparse_tag = WIM_IO_REPARSE_TAG_SYMLINK;
 		if (inode_set_symlink(dentry->d_inode, to,
-				      ctx->wim->lookup_table, NULL))
+				      wimfs_ctx->wim->lookup_table, NULL))
 		{
-			unlink_dentry(dentry);
-			free_dentry(dentry);
+			remove_dentry(dentry, wimfs_ctx->wim->lookup_table);
 			ret = -ENOMEM;
 		}
 	}
@@ -2205,13 +2254,13 @@ static int wimfs_utimens(const char *path, const struct timespec tv[2])
 		if (tv[0].tv_nsec == UTIME_NOW)
 			inode->i_last_access_time = get_wim_timestamp();
 		else
-			inode->i_last_access_time = timespec_to_wim_timestamp(&tv[0]);
+			inode->i_last_access_time = timespec_to_wim_timestamp(tv[0]);
 	}
 	if (tv[1].tv_nsec != UTIME_OMIT) {
 		if (tv[1].tv_nsec == UTIME_NOW)
 			inode->i_last_write_time = get_wim_timestamp();
 		else
-			inode->i_last_write_time = timespec_to_wim_timestamp(&tv[1]);
+			inode->i_last_write_time = timespec_to_wim_timestamp(tv[1]);
 	}
 	return 0;
 }
@@ -2260,15 +2309,17 @@ static int wimfs_write(const char *path, const char *buf, size_t size,
 	if (ret == -1)
 		return -errno;
 
-	now = get_wim_timestamp();
-	fd->f_inode->i_last_write_time = now;
-	fd->f_inode->i_last_access_time = now;
+	/* Update timestamps */
+	touch_inode(fd->f_inode);
 	return ret;
 }
 
 static struct fuse_operations wimfs_operations = {
+#if 0
 	.access      = wimfs_access,
+#endif
 	.chmod       = wimfs_chmod,
+	.chown       = wimfs_chown,
 	.destroy     = wimfs_destroy,
 #if 0
 	.fallocate   = wimfs_fallocate,
@@ -2329,7 +2380,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
 				 unsigned num_additional_swms,
 				 const char *staging_dir)
 {
-	int argc = 0;
+	int argc;
 	char *argv[16];
 	int ret;
 	char *dir_copy;
@@ -2413,7 +2464,8 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
 	ctx.wim = wim;
 	ctx.mount_flags = mount_flags;
 	ctx.image_inode_list = &imd->inode_list;
-
+	ctx.default_uid = getuid();
+	ctx.default_gid = getgid();
 	if (mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_WINDOWS)
 		ctx.default_lookup_flags = LOOKUP_FLAG_ADS_OK;
 
@@ -2429,6 +2481,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
 	if (!dir_copy)
 		goto out_free_message_queue_names;
 
+	argc = 0;
 	argv[argc++] = "imagex";
 	argv[argc++] = dir_copy;
 
@@ -2441,7 +2494,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
 
 	/*
 	 * We provide the use_ino option to the FUSE mount because we are going
-	 * to assign inode numbers oursides. */
+	 * to assign inode numbers ourselves. */
 	char optstring[256] =
 		"use_ino"
 		",subtype=wimfs"
@@ -2449,6 +2502,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
 #if FUSE_MAJOR_VERSION > 2 || (FUSE_MAJOR_VERSION == 2 && FUSE_MINOR_VERSION >= 8)
 		",hard_remove"
 #endif
+		",default_permissions"
 		;
 	argv[argc++] = "-o";
 	argv[argc++] = optstring;
@@ -2461,6 +2515,8 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
 		/* Read-only mount */
 		strcat(optstring, ",ro");
 	}
+	if (mount_flags & WIMLIB_MOUNT_FLAG_ALLOW_OTHER)
+		strcat(optstring, ",allow_other");
 	argv[argc] = NULL;
 
 #ifdef ENABLE_DEBUG
@@ -2498,8 +2554,23 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
 
 	DEBUG("Returned from fuse_main() (ret = %d)", ret);
 
-	if (ret)
+	if (ret) {
 		ret = WIMLIB_ERR_FUSE;
+	} else {
+		if (ctx.have_status)
+			ret = ctx.status;
+		else
+			ret = WIMLIB_ERR_TIMEOUT;
+	}
+	if (ctx.daemon_to_unmount_mq != (mqd_t)(-1)) {
+		send_unmount_finished_msg(ctx.daemon_to_unmount_mq, ret);
+		close_message_queues(&ctx);
+	}
+
+	/* Try to delete the staging directory if a deletion wasn't yet
+	 * attempted due to an earlier error */
+	if (ctx.staging_dir_name)
+		delete_staging_dir(&ctx);
 out_free_dir_copy:
 	FREE(dir_copy);
 out_unlock:
@@ -2570,8 +2641,12 @@ out:
 
 static inline int mount_unsupported_error()
 {
+#if defined(__WIN32__)
+	ERROR("Sorry-- Mounting WIM images is not supported on Windows!");
+#else
 	ERROR("wimlib was compiled with --without-fuse, which disables support "
 	      "for mounting WIMs.");
+#endif
 	return WIMLIB_ERR_UNSUPPORTED;
 }