]> wimlib.net Git - wimlib/blobdiff - src/mount_image.c
rebuild_wim(): Fix use of freed memory
[wimlib] / src / mount_image.c
index 74b751c95047e28cc1053752c08e3cd79d7dac63..28a807d7afdfd7d26537a57a97d99e5203564841 100644 (file)
 #include <ftw.h>
 #include <mqueue.h>
 #include <utime.h>
+#include <signal.h>
 
 #ifdef ENABLE_XATTR
 #include <attr/xattr.h>
 #endif
 
+#define MSG_VERSION_TOO_HIGH   -1
+#define MSG_BREAK_LOOP         -2
+
 /* File descriptor to a file open on the WIM filesystem. */
-struct wimlib_fd {
-       struct inode *f_inode;
-       struct lookup_table_entry *f_lte;
+struct wimfs_fd {
+       struct wim_inode *f_inode;
+       struct wim_lookup_table_entry *f_lte;
        int staging_fd;
        u16 idx;
        u32 stream_id;
@@ -76,23 +80,28 @@ struct wimfs_context {
        /* Flags passed to wimlib_mount(). */
        int mount_flags;
 
+       /* Default flags to use when looking up a WIM dentry (depends on whether
+        * the Windows interface to alternate data streams is being used or
+        * not). */
        int default_lookup_flags;
 
-       /* Next inode number to be assigned. */
+       /* Next inode number to be assigned.  Note: I didn't bother with a
+        * bitmap of free inode numbers since this isn't even a "real"
+        * filesystem anyway. */
        u64 next_ino;
 
-       /* List of lookup table entries in the staging directory */
+       /* List of lookup table entries for files in the staging directory */
        struct list_head staging_list;
 
        /* List of inodes in the mounted image */
        struct hlist_head *image_inode_list;
 
-       /* Name and message queue descriptors for message queues between the filesystem
-        * daemon process and the unmount process.  These are used when the filesystem
-        * is unmounted and the process running wimlib_mount() (i.e. the `imagex
-        * unmount' command) needs to communicate with the filesystem daemon running
-        * fuse_main() (i.e. that spawned by the `imagex mount' or `imagex mountrw'
-        * commands */
+       /* Name and message queue descriptors for message queues between the
+        * filesystem daemon process and the unmount process.  These are used
+        * when the filesystem is unmounted and the process running
+        * wimlib_unmount_image() (i.e. the `imagex unmount' command) needs to
+        * communicate with the filesystem daemon running fuse_main() (i.e. the
+        * daemon created by the `imagex mount' or `imagex mountrw' commands */
        char *unmount_to_daemon_mq_name;
        char *daemon_to_unmount_mq_name;
        mqd_t unmount_to_daemon_mq;
@@ -144,11 +153,11 @@ static inline int flags_writable(int open_flags)
  *
  * Return 0 iff successful or error code if unsuccessful.
  */
-static int alloc_wimlib_fd(struct inode *inode,
-                          u32 stream_id,
-                          struct lookup_table_entry *lte,
-                          struct wimlib_fd **fd_ret,
-                          bool readonly)
+static int alloc_wimfs_fd(struct wim_inode *inode,
+                         u32 stream_id,
+                         struct wim_lookup_table_entry *lte,
+                         struct wimfs_fd **fd_ret,
+                         bool readonly)
 {
        static const u16 fds_per_alloc = 8;
        static const u16 max_fds = 0xffff;
@@ -156,36 +165,37 @@ static int alloc_wimlib_fd(struct inode *inode,
 
        pthread_mutex_lock(&inode->i_mutex);
 
-       DEBUG("Allocating fd for stream ID %u from inode %lx (open = %u, allocated = %u)",
-             stream_id, inode->ino, inode->num_opened_fds,
-             inode->num_allocated_fds);
+       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);
 
-       if (inode->num_opened_fds == inode->num_allocated_fds) {
-               struct wimlib_fd **fds;
+       if (inode->i_num_opened_fds == inode->i_num_allocated_fds) {
+               struct wimfs_fd **fds;
                u16 num_new_fds;
 
-               if (inode->num_allocated_fds == max_fds) {
+               if (inode->i_num_allocated_fds == max_fds) {
                        ret = -EMFILE;
                        goto out;
                }
                num_new_fds = min(fds_per_alloc,
-                                 max_fds - inode->num_allocated_fds);
+                                 max_fds - inode->i_num_allocated_fds);
 
-               fds = REALLOC(inode->fds,
-                             (inode->num_allocated_fds + num_new_fds) *
-                               sizeof(inode->fds[0]));
+               fds = REALLOC(inode->i_fds,
+                             (inode->i_num_allocated_fds + num_new_fds) *
+                               sizeof(inode->i_fds[0]));
                if (!fds) {
                        ret = -ENOMEM;
                        goto out;
                }
-               memset(&fds[inode->num_allocated_fds], 0,
+               memset(&fds[inode->i_num_allocated_fds], 0,
                       num_new_fds * sizeof(fds[0]));
-               inode->fds = fds;
-               inode->num_allocated_fds += num_new_fds;
+               inode->i_fds = fds;
+               inode->i_num_allocated_fds += num_new_fds;
        }
        for (u16 i = 0; ; i++) {
-               if (!inode->fds[i]) {
-                       struct wimlib_fd *fd = CALLOC(1, sizeof(*fd));
+               if (!inode->i_fds[i]) {
+                       struct wimfs_fd *fd = CALLOC(1, sizeof(*fd));
                        if (!fd) {
                                ret = -ENOMEM;
                                break;
@@ -196,8 +206,8 @@ static int alloc_wimlib_fd(struct inode *inode,
                        fd->idx        = i;
                        fd->stream_id  = stream_id;
                        *fd_ret        = fd;
-                       inode->fds[i]  = fd;
-                       inode->num_opened_fds++;
+                       inode->i_fds[i]  = fd;
+                       inode->i_num_opened_fds++;
                        if (lte && !readonly)
                                lte->num_opened_fds++;
                        DEBUG("Allocated fd (idx = %u)", fd->idx);
@@ -210,20 +220,20 @@ out:
        return ret;
 }
 
-static void inode_put_fd(struct inode *inode, struct wimlib_fd *fd)
+static void inode_put_fd(struct wim_inode *inode, struct wimfs_fd *fd)
 {
        wimlib_assert(inode != NULL);
 
        pthread_mutex_lock(&inode->i_mutex);
 
        wimlib_assert(fd->f_inode == inode);
-       wimlib_assert(inode->num_opened_fds != 0);
-       wimlib_assert(fd->idx < inode->num_allocated_fds);
-       wimlib_assert(inode->fds[fd->idx] == fd);
+       wimlib_assert(inode->i_num_opened_fds != 0);
+       wimlib_assert(fd->idx < inode->i_num_allocated_fds);
+       wimlib_assert(inode->i_fds[fd->idx] == fd);
 
-       inode->fds[fd->idx] = NULL;
+       inode->i_fds[fd->idx] = NULL;
        FREE(fd);
-       if (--inode->num_opened_fds == 0 && inode->link_count == 0) {
+       if (--inode->i_num_opened_fds == 0 && inode->i_nlink == 0) {
                pthread_mutex_unlock(&inode->i_mutex);
                free_inode(inode);
        } else {
@@ -231,7 +241,7 @@ static void inode_put_fd(struct inode *inode, struct wimlib_fd *fd)
        }
 }
 
-static int lte_put_fd(struct lookup_table_entry *lte, struct wimlib_fd *fd)
+static int lte_put_fd(struct wim_lookup_table_entry *lte, struct wimfs_fd *fd)
 {
        wimlib_assert(fd->f_lte == lte);
 
@@ -253,12 +263,12 @@ static int lte_put_fd(struct lookup_table_entry *lte, struct wimlib_fd *fd)
 }
 
 /* Close a file descriptor. */
-static int close_wimlib_fd(struct wimlib_fd *fd)
+static int close_wimfs_fd(struct wimfs_fd *fd)
 {
        int ret;
-       DEBUG("Closing fd (inode = %lu, opened = %u, allocated = %u)",
-             fd->f_inode->ino, fd->f_inode->num_opened_fds,
-             fd->f_inode->num_allocated_fds);
+       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);
        if (ret != 0)
                return ret;
@@ -267,16 +277,25 @@ static int close_wimlib_fd(struct wimlib_fd *fd)
        return 0;
 }
 
+/*
+ * 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,
-                        struct dentry **dentry_ret)
+                        struct wim_dentry **dentry_ret)
 {
-       struct dentry *parent;
-       struct dentry *new;
+       struct wim_dentry *parent;
+       struct wim_dentry *new;
        const char *basename;
 
        parent = get_parent_dentry(ctx->wim, path);
        if (!parent)
-               return -ENOENT;
+               return -errno;
 
        if (!dentry_is_directory(parent))
                return -ENOTDIR;
@@ -287,17 +306,18 @@ static int create_dentry(struct wimfs_context *ctx, const char *path,
 
        new = new_dentry_with_inode(basename);
        if (!new)
-               return -ENOMEM;
+               return -errno;
 
-       new->d_inode->resolved = 1;
-       new->d_inode->ino = ctx->next_ino++;
+       new->d_inode->i_resolved = 1;
+       new->d_inode->i_ino = ctx->next_ino++;
        dentry_add_child(parent, new);
-       hlist_add_head(&new->d_inode->hlist, ctx->image_inode_list);
+       hlist_add_head(&new->d_inode->i_hlist, ctx->image_inode_list);
        *dentry_ret = new;
        return 0;
 }
 
-/* Remove a dentry; i.e. remove a reference to the corresponding inode.
+/* Remove a dentry from a mounted WIM image; i.e. remove an alias for the
+ * corresponding inode.
  *
  * If there are no remaining references to the inode either through dentries or
  * open file descriptors, the inode is freed.  Otherwise, the inode is not
@@ -305,17 +325,17 @@ static int create_dentry(struct wimfs_context *ctx, const char *path,
  *
  * Either way, all lookup table entries referenced by the inode have their
  * reference count decremented.  If a lookup table entry has no open file
- * descriptors and no references remaining, it is freed, and the staging file is
- * unlinked.
+ * descriptors and no references remaining, it is freed, and the corresponding
+ * staging file is unlinked.
  */
-static void remove_dentry(struct dentry *dentry,
-                         struct lookup_table *lookup_table)
+static void remove_dentry(struct wim_dentry *dentry,
+                         struct wim_lookup_table *lookup_table)
 {
-       struct inode *inode = dentry->d_inode;
-       struct lookup_table_entry *lte;
+       struct wim_inode *inode = dentry->d_inode;
+       struct wim_lookup_table_entry *lte;
        unsigned i;
 
-       for (i = 0; i <= inode->num_ads; i++) {
+       for (i = 0; i <= inode->i_num_ads; i++) {
                lte = inode_stream_lte_resolved(inode, i);
                if (lte)
                        lte_decrement_refcnt(lte, lookup_table);
@@ -324,13 +344,14 @@ static void remove_dentry(struct dentry *dentry,
        put_dentry(dentry);
 }
 
-/* Transfers file attributes from a struct inode to a `stat' buffer.
+/* 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.
  * For a named data stream, everything returned is the same as the unnamed data
  * stream except possibly the size and block count. */
-static int inode_to_stbuf(const struct inode *inode,
-                         struct lookup_table_entry *lte, struct stat *stbuf)
+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;
@@ -339,8 +360,8 @@ static int inode_to_stbuf(const struct inode *inode,
        else
                stbuf->st_mode = S_IFREG | 0755;
 
-       stbuf->st_ino   = (ino_t)inode->ino;
-       stbuf->st_nlink = inode->link_count;
+       stbuf->st_ino   = (ino_t)inode->i_ino;
+       stbuf->st_nlink = inode->i_nlink;
        stbuf->st_uid   = getuid();
        stbuf->st_gid   = getgid();
 
@@ -360,9 +381,9 @@ static int inode_to_stbuf(const struct inode *inode,
                stbuf->st_size = 0;
        }
 
-       stbuf->st_atime   = wim_timestamp_to_unix(inode->last_access_time);
-       stbuf->st_mtime   = wim_timestamp_to_unix(inode->last_write_time);
-       stbuf->st_ctime   = wim_timestamp_to_unix(inode->creation_time);
+       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;
        return 0;
 }
@@ -371,12 +392,15 @@ static int inode_to_stbuf(const struct inode *inode,
  * writing.
  *
  * @name_ret: A location into which the a pointer to the newly allocated name of
- *                     the staging file is stored.
- * @return:  The file descriptor for the new file.  Returns -1 and sets errno on
- *             error, for any reason possible from the creat() function.
+ *           the staging file is stored.
+ *
+ * @ctx:      Context for the WIM filesystem; this provides the name of the
+ *           staging directory.
+ *
+ * On success, returns the file descriptor for the staging file, opened for
+ * writing.  On failure, returns -1 and sets errno.
  */
-static int create_staging_file(char **name_ret, int open_flags,
-                              struct wimfs_context *ctx)
+static int create_staging_file(char **name_ret, struct wimfs_context *ctx)
 {
        size_t name_len;
        char *name;
@@ -384,7 +408,9 @@ static int create_staging_file(char **name_ret, int open_flags,
        int fd;
        int errno_save;
 
-       name_len = ctx->staging_dir_name_len + 1 + SHA1_HASH_SIZE;
+       static const size_t STAGING_FILE_NAME_LEN = 20;
+
+       name_len = ctx->staging_dir_name_len + 1 + STAGING_FILE_NAME_LEN;
        name = MALLOC(name_len + 1);
        if (!name) {
                errno = ENOMEM;
@@ -396,7 +422,7 @@ static int create_staging_file(char **name_ret, int open_flags,
                memcpy(name, ctx->staging_dir_name, ctx->staging_dir_name_len);
                name[ctx->staging_dir_name_len] = '/';
                randomize_char_array_with_alnum(name + ctx->staging_dir_name_len + 1,
-                                               SHA1_HASH_SIZE);
+                                               STAGING_FILE_NAME_LEN);
                name[name_len] = '\0';
 
 
@@ -404,15 +430,14 @@ static int create_staging_file(char **name_ret, int open_flags,
         * existing file, and try again if so  */
        } while (stat(name, &stbuf) == 0);
 
-       if (errno != ENOENT)
-               /* other error! */
+       if (errno != ENOENT) /* other error?! */
                return -1;
 
        /* doesn't exist--- ok */
 
        DEBUG("Creating staging file `%s'", name);
 
-       fd = open(name, open_flags | O_CREAT | O_TRUNC, 0600);
+       fd = open(name, O_WRONLY | O_CREAT | O_EXCL, 0600);
        if (fd == -1) {
                errno_save = errno;
                FREE(name);
@@ -442,20 +467,20 @@ static int create_staging_file(char **name_ret, int open_flags,
  *
  * Returns 0 on success or a negative error code on failure.
  */
-static int extract_resource_to_staging_dir(struct inode *inode,
+static int extract_resource_to_staging_dir(struct wim_inode *inode,
                                           u32 stream_id,
-                                          struct lookup_table_entry **lte,
+                                          struct wim_lookup_table_entry **lte,
                                           off_t size,
                                           struct wimfs_context *ctx)
 {
        char *staging_file_name;
        int ret;
        int fd;
-       struct lookup_table_entry *old_lte, *new_lte;
+       struct wim_lookup_table_entry *old_lte, *new_lte;
        off_t extract_size;
 
        DEBUG("Extracting resource to staging dir: inode %"PRIu64", "
-             "stream id %"PRIu32, inode->ino, stream_id);
+             "stream id %"PRIu32, inode->i_ino, stream_id);
 
        old_lte = *lte;
 
@@ -463,7 +488,7 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                      old_lte->resource_location != RESOURCE_IN_STAGING_FILE);
 
        /* Create the staging file */
-       fd = create_staging_file(&staging_file_name, O_WRONLY, ctx);
+       fd = create_staging_file(&staging_file_name, ctx);
        if (fd == -1)
                return -errno;
 
@@ -496,7 +521,7 @@ static int extract_resource_to_staging_dir(struct inode *inode,
        /* Now deal with the lookup table entries.  We may be able to re-use the
         * existing entry, but we may have to create a new one instead. */
 
-       if (old_lte && inode->link_count == old_lte->refcnt) {
+       if (old_lte && inode->i_nlink == old_lte->refcnt) {
                /* The reference count of the existing lookup table entry is the
                 * same as the link count of the inode that contains the stream
                 * we're opening.  Therefore, ALL the references to the lookup
@@ -512,10 +537,10 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                         * the inode containing a stream we're opening.
                         * Therefore, we need to split the lookup table entry.
                         */
-                       wimlib_assert(old_lte->refcnt > inode->link_count);
+                       wimlib_assert(old_lte->refcnt > inode->i_nlink);
                        DEBUG("Splitting lookup table entry "
-                             "(inode->link_count = %u, old_lte->refcnt = %u)",
-                             inode->link_count, old_lte->refcnt);
+                             "(inode->i_nlink = %u, old_lte->refcnt = %u)",
+                             inode->i_nlink, old_lte->refcnt);
                }
 
                new_lte = new_lookup_table_entry();
@@ -535,8 +560,8 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                 * file descriptors to the new lookup table entry.  If there's
                 * an old lookup table entry, this number needs to be subtracted
                 * from the fd's opened to the old entry. */
-               for (u16 i = 0, j = 0; j < inode->num_opened_fds; i++) {
-                       struct wimlib_fd *fd = inode->fds[i];
+               for (u16 i = 0, j = 0; j < inode->i_num_opened_fds; i++) {
+                       struct wimfs_fd *fd = inode->i_fds[i];
                        if (fd) {
                                if (fd->stream_id == stream_id) {
                                        wimlib_assert(fd->f_lte == old_lte);
@@ -556,22 +581,22 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                      new_lte->num_opened_fds);
                if (old_lte) {
                        old_lte->num_opened_fds -= new_lte->num_opened_fds;
-                       old_lte->refcnt -= inode->link_count;
+                       old_lte->refcnt -= inode->i_nlink;
                }
        }
 
-       new_lte->refcnt                       = inode->link_count;
+       new_lte->refcnt                       = inode->i_nlink;
        new_lte->resource_location            = RESOURCE_IN_STAGING_FILE;
        new_lte->staging_file_name            = staging_file_name;
        new_lte->lte_inode                    = inode;
        random_hash(new_lte->hash);
 
        if (stream_id == 0)
-               inode->lte = new_lte;
+               inode->i_lte = new_lte;
        else
-               for (u16 i = 0; i < inode->num_ads; i++)
-                       if (inode->ads_entries[i].stream_id == stream_id)
-                               inode->ads_entries[i].lte = new_lte;
+               for (u16 i = 0; i < inode->i_num_ads; i++)
+                       if (inode->i_ads_entries[i].stream_id == stream_id)
+                               inode->i_ads_entries[i].lte = new_lte;
 
        lookup_table_insert(ctx->wim->lookup_table, new_lte);
        list_add(&new_lte->staging_list, &ctx->staging_list);
@@ -579,7 +604,7 @@ static int extract_resource_to_staging_dir(struct inode *inode,
        return 0;
 out_revert_fd_changes:
        for (u16 i = 0, j = 0; j < new_lte->num_opened_fds; i++) {
-               struct wimlib_fd *fd = inode->fds[i];
+               struct wimfs_fd *fd = inode->i_fds[i];
                if (fd && fd->stream_id == stream_id && fd->f_lte == new_lte) {
                        fd->f_lte = old_lte;
                        if (fd->staging_fd != -1) {
@@ -676,7 +701,6 @@ static int remove_file_or_directory(const char *fpath, const struct stat *sb,
        }
 }
 
-
 /*
  * Deletes the staging directory and all the files contained in it.
  */
@@ -690,6 +714,108 @@ static int delete_staging_dir(struct wimfs_context *ctx)
        return ret;
 }
 
+static void inode_update_lte_ptr(struct wim_inode *inode,
+                                struct wim_lookup_table_entry *old_lte,
+                                struct wim_lookup_table_entry *new_lte)
+{
+       if (inode->i_lte == old_lte) {
+               inode->i_lte = new_lte;
+       } else {
+               for (unsigned i = 0; i < inode->i_num_ads; i++) {
+                       if (inode->i_ads_entries[i].lte == old_lte) {
+                               inode->i_ads_entries[i].lte = new_lte;
+                               break;
+                       }
+               }
+       }
+}
+
+static int update_lte_of_staging_file(struct wim_lookup_table_entry *lte,
+                                     struct wim_lookup_table *table)
+{
+       struct wim_lookup_table_entry *duplicate_lte;
+       int ret;
+       u8 hash[SHA1_HASH_SIZE];
+       struct stat stbuf;
+
+       ret = sha1sum(lte->staging_file_name, hash);
+       if (ret != 0)
+               return ret;
+       lookup_table_unlink(table, lte);
+       duplicate_lte = __lookup_resource(table, hash);
+       if (duplicate_lte) {
+               /* Merge duplicate lookup table entries */
+               duplicate_lte->refcnt += lte->refcnt;
+               inode_update_lte_ptr(lte->lte_inode, lte, duplicate_lte);
+               free_lookup_table_entry(lte);
+       } else {
+               if (stat(lte->staging_file_name, &stbuf) != 0) {
+                       ERROR_WITH_ERRNO("Failed to stat `%s'", lte->staging_file_name);
+                       return WIMLIB_ERR_STAT;
+               }
+               if (stbuf.st_size == 0) {
+                       /* Zero-length stream.  No lookup table entry needed. */
+                       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);
+                       lte->resource_entry.original_size = stbuf.st_size;
+                       lte->resource_entry.size = stbuf.st_size;
+                       lte->resource_location = RESOURCE_IN_FILE_ON_DISK;
+                       lte->file_on_disk_fp = NULL;
+                       copy_hash(lte->hash, hash);
+                       lookup_table_insert(table, lte);
+               }
+       }
+       return 0;
+}
+
+static int inode_close_fds(struct wim_inode *inode)
+{
+       u16 num_opened_fds = inode->i_num_opened_fds;
+       for (u16 i = 0, j = 0; j < num_opened_fds; i++) {
+               struct wimfs_fd *fd = inode->i_fds[i];
+               if (fd) {
+                       wimlib_assert(fd->f_inode == inode);
+                       int ret = close_wimfs_fd(fd);
+                       if (ret != 0)
+                               return ret;
+                       j++;
+               }
+       }
+       return 0;
+}
+
+/* Overwrites the WIM file, with changes saved. */
+static int rebuild_wim(struct wimfs_context *ctx, int write_flags,
+                      wimlib_progress_func_t progress_func)
+{
+       int ret;
+       struct wim_lookup_table_entry *lte, *tmp;
+       WIMStruct *w = ctx->wim;
+
+       DEBUG("Closing all staging file descriptors.");
+       list_for_each_entry_safe(lte, tmp, &ctx->staging_list, staging_list) {
+               ret = inode_close_fds(lte->lte_inode);
+               if (ret != 0)
+                       return ret;
+       }
+
+       DEBUG("Calculating SHA1 checksums for all new staging files.");
+       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;
+       }
+
+       xml_update_image_info(w, w->current_image);
+       ret = wimlib_overwrite(w, write_flags, 0, progress_func);
+       if (ret != 0)
+               ERROR("Failed to commit changes to mounted WIM image");
+       return ret;
+}
+
+
 
 /* Simple function that returns the concatenation of 2 strings. */
 static char *strcat_dup(const char *s1, const char *s2, size_t max_len)
@@ -764,8 +890,7 @@ static void free_message_queue_names(struct wimfs_context *ctx)
  * Opens two POSIX message queue: one for sending messages from the unmount
  * process to the daemon process, and one to go the other way.  The names of the
  * message queues, which must be system-wide unique, are be based on the mount
- * point.  (There of course is still a possibility of a collision if one were to
- * unmount two identically named directories simultaneously...)
+ * point.
  *
  * @daemon specifies whether the calling process is the filesystem daemon or the
  * unmount process.
@@ -866,6 +991,7 @@ static void unlink_message_queues(struct wimfs_context *ctx)
 /* Closes the message queues, which are allocated in static variables */
 static void close_message_queues(struct wimfs_context *ctx)
 {
+       DEBUG("Closing message queues");
        mq_close(ctx->unmount_to_daemon_mq);
        ctx->unmount_to_daemon_mq = (mqd_t)(-1);
        mq_close(ctx->daemon_to_unmount_mq);
@@ -873,277 +999,386 @@ static void close_message_queues(struct wimfs_context *ctx)
        unlink_message_queues(ctx);
 }
 
-struct unmount_message_header {
+
+struct unmount_msg_hdr {
        u32 min_version;
        u32 cur_version;
-       u32 data_size;
+       u32 msg_type;
+       u32 msg_size;
+} PACKED;
+
+struct msg_unmount_request {
+       struct unmount_msg_hdr hdr;
+       u32 unmount_flags;
+       u8 want_progress_messages;
+} PACKED;
+
+struct msg_daemon_info {
+       struct unmount_msg_hdr hdr;
+       pid_t daemon_pid;
+       u32 mount_flags;
+} PACKED;
+
+struct msg_unmount_finished {
+       struct unmount_msg_hdr hdr;
+       int32_t status;
+} PACKED;
+
+struct msg_write_streams_progress {
+       struct unmount_msg_hdr hdr;
+       union wimlib_progress_info info;
+} PACKED;
+
+enum {
+       MSG_TYPE_UNMOUNT_REQUEST,
+       MSG_TYPE_DAEMON_INFO,
+       MSG_TYPE_WRITE_STREAMS_PROGRESS,
+       MSG_TYPE_UNMOUNT_FINISHED,
+       MSG_TYPE_MAX,
 };
 
-struct unmount_request {
-       struct unmount_message_header hdr;
-       struct {
-               u32 unmount_flags;
-       } data;
+struct msg_handler_context_hdr {
+       int timeout_seconds;
 };
 
-struct unmount_response {
-       struct unmount_message_header hdr;
-       struct {
-               int32_t status;
-               u32 mount_flags;
-       } data;
+struct unmount_msg_handler_context {
+       struct msg_handler_context_hdr hdr;
+       pid_t daemon_pid;
+       int mount_flags;
+       int status;
+       wimlib_progress_func_t progress_func;
 };
 
-static void inode_update_lte_ptr(struct inode *inode,
-                                struct lookup_table_entry *old_lte,
-                                struct lookup_table_entry *new_lte)
-{
-       if (inode->lte == old_lte) {
-               inode->lte = new_lte;
-       } else {
-               for (unsigned i = 0; i < inode->num_ads; i++) {
-                       if (inode->ads_entries[i].lte == old_lte) {
-                               inode->ads_entries[i].lte = new_lte;
-                               break;
-                       }
-               }
+struct daemon_msg_handler_context {
+       struct msg_handler_context_hdr hdr;
+       struct wimfs_context *wimfs_ctx;
+};
+
+static int send_unmount_request_msg(mqd_t mq, int unmount_flags,
+                                   u8 want_progress_messages)
+{
+       DEBUG("Sending unmount request msg");
+       struct msg_unmount_request msg = {
+               .hdr = {
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 1),
+                       .cur_version = WIMLIB_VERSION_CODE,
+                       .msg_type    = MSG_TYPE_UNMOUNT_REQUEST,
+                       .msg_size    = sizeof(msg),
+               },
+               .unmount_flags = unmount_flags,
+               .want_progress_messages = want_progress_messages,
+       };
+
+       if (mq_send(mq, (void*)&msg, sizeof(msg), 1)) {
+               ERROR_WITH_ERRNO("Failed to communicate with filesystem daemon");
+               return WIMLIB_ERR_MQUEUE;
        }
+       return 0;
 }
 
-static int update_lte_of_staging_file(struct lookup_table_entry *lte,
-                                     struct lookup_table *table)
+static int send_daemon_info_msg(mqd_t mq, pid_t pid, int mount_flags)
 {
-       struct lookup_table_entry *duplicate_lte;
-       int ret;
-       u8 hash[SHA1_HASH_SIZE];
-       struct stat stbuf;
+       DEBUG("Sending daemon info msg (pid = %d, mount_flags=%x)",
+             pid, mount_flags);
 
-       ret = sha1sum(lte->staging_file_name, hash);
-       if (ret != 0)
-               return ret;
-       lookup_table_unlink(table, lte);
-       duplicate_lte = __lookup_resource(table, hash);
-       if (duplicate_lte) {
-               /* Merge duplicate lookup table entries */
-               duplicate_lte->refcnt += lte->refcnt;
-               inode_update_lte_ptr(lte->lte_inode, lte, duplicate_lte);
-               free_lookup_table_entry(lte);
-       } else {
-               if (stat(lte->staging_file_name, &stbuf) != 0) {
-                       ERROR_WITH_ERRNO("Failed to stat `%s'", lte->staging_file_name);
-                       return WIMLIB_ERR_STAT;
-               }
-               if (stbuf.st_size == 0) {
-                       /* Zero-length stream.  No lookup table entry needed. */
-                       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);
-                       lte->resource_entry.original_size = stbuf.st_size;
-                       lte->resource_entry.size = stbuf.st_size;
-                       lte->resource_location = RESOURCE_IN_FILE_ON_DISK;
-                       lte->file_on_disk_fp = NULL;
-                       copy_hash(lte->hash, hash);
-                       lookup_table_insert(table, lte);
-               }
+       struct msg_daemon_info msg = {
+               .hdr = {
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 1),
+                       .cur_version = WIMLIB_VERSION_CODE,
+                       .msg_type = MSG_TYPE_DAEMON_INFO,
+                       .msg_size = sizeof(msg),
+               },
+               .daemon_pid = pid,
+               .mount_flags = mount_flags,
+       };
+       if (mq_send(mq, (void*)&msg, sizeof(msg), 1)) {
+               ERROR_WITH_ERRNO("Failed to send daemon info to unmount process");
+               return WIMLIB_ERR_MQUEUE;
        }
        return 0;
 }
 
-static int inode_close_fds(struct inode *inode)
+static void send_unmount_finished_msg(mqd_t mq, int status)
 {
-       u16 num_opened_fds = inode->num_opened_fds;
-       for (u16 i = 0, j = 0; j < num_opened_fds; i++) {
-               struct wimlib_fd *fd = inode->fds[i];
-               if (fd) {
-                       wimlib_assert(fd->f_inode == inode);
-                       int ret = close_wimlib_fd(fd);
-                       if (ret != 0)
-                               return ret;
-                       j++;
+       DEBUG("Sending unmount finished msg");
+       struct msg_unmount_finished msg = {
+               .hdr = {
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 1),
+                       .cur_version = WIMLIB_VERSION_CODE,
+                       .msg_type = MSG_TYPE_UNMOUNT_FINISHED,
+                       .msg_size = sizeof(msg),
+               },
+               .status = status,
+       };
+       if (mq_send(mq, (void*)&msg, sizeof(msg), 1))
+               ERROR_WITH_ERRNO("Failed to send status to unmount process");
+}
+
+static int unmount_progress_func(enum wimlib_progress_msg msg,
+                                const union wimlib_progress_info *info)
+{
+       if (msg == WIMLIB_PROGRESS_MSG_WRITE_STREAMS) {
+               struct msg_write_streams_progress msg = {
+                       .hdr = {
+                               .min_version = WIMLIB_MAKEVERSION(1, 2, 1),
+                               .cur_version = WIMLIB_VERSION_CODE,
+                               .msg_type = MSG_TYPE_WRITE_STREAMS_PROGRESS,
+                               .msg_size = sizeof(msg),
+                       },
+                       .info = *info,
+               };
+               if (mq_send(wimfs_get_context()->daemon_to_unmount_mq,
+                           (void*)&msg, sizeof(msg), 1))
+               {
+                       ERROR_WITH_ERRNO("Failed to send progress information "
+                                        "to unmount process");
                }
        }
        return 0;
 }
 
-/* Overwrites the WIM file, with changes saved. */
-static int rebuild_wim(struct wimfs_context *ctx, int write_flags)
+static int msg_unmount_request_handler(const void *_msg, void *_handler_ctx)
 {
+       const struct msg_unmount_request *msg = _msg;
+       struct daemon_msg_handler_context *handler_ctx = _handler_ctx;
+       struct wimfs_context *wimfs_ctx;
+       int status = 0;
        int ret;
-       struct lookup_table_entry *lte, *tmp;
-       WIMStruct *w = ctx->wim;
+       int unmount_flags;
+       wimlib_progress_func_t progress_func;
 
-       DEBUG("Closing all staging file descriptors.");
-       list_for_each_entry_safe(lte, tmp, &ctx->staging_list, staging_list) {
-               ret = inode_close_fds(lte->lte_inode);
-               if (ret != 0)
-                       return ret;
+       DEBUG("Handling unmount request msg");
+
+       wimfs_ctx = handler_ctx->wimfs_ctx;
+       if (msg->hdr.msg_size < sizeof(*msg)) {
+               status = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+               goto out;
        }
 
-       DEBUG("Calculating SHA1 checksums for all new staging files.");
-       list_for_each_entry(lte, &ctx->staging_list, staging_list) {
-               ret = update_lte_of_staging_file(lte, w->lookup_table);
-               if (ret != 0)
-                       return ret;
+       unmount_flags = msg->unmount_flags;
+       if (msg->want_progress_messages)
+               progress_func = unmount_progress_func;
+       else
+               progress_func = NULL;
+
+       ret = send_daemon_info_msg(wimfs_ctx->daemon_to_unmount_mq, getpid(),
+                                  wimfs_ctx->mount_flags);
+       if (ret != 0) {
+               status = ret;
+               goto out;
        }
 
-       xml_update_image_info(w, w->current_image);
-       ret = wimlib_overwrite(w, write_flags, 0, NULL);
-       if (ret != 0)
-               ERROR("Failed to commit changes to mounted WIM image");
-       return ret;
+       if (wimfs_ctx->mount_flags & WIMLIB_MOUNT_FLAG_READWRITE) {
+               if (unmount_flags & WIMLIB_UNMOUNT_FLAG_COMMIT) {
+                       int write_flags = 0;
+                       if (unmount_flags & WIMLIB_UNMOUNT_FLAG_CHECK_INTEGRITY)
+                               write_flags |= WIMLIB_WRITE_FLAG_CHECK_INTEGRITY;
+                       if (unmount_flags & WIMLIB_UNMOUNT_FLAG_REBUILD)
+                               write_flags |= WIMLIB_WRITE_FLAG_REBUILD;
+                       if (unmount_flags & WIMLIB_UNMOUNT_FLAG_RECOMPRESS)
+                               write_flags |= WIMLIB_WRITE_FLAG_RECOMPRESS;
+                       status = rebuild_wim(wimfs_ctx, write_flags,
+                                            progress_func);
+               }
+       } else {
+               DEBUG("Read-only mount");
+               status = 0;
+       }
+
+out:
+       if (wimfs_ctx->mount_flags & WIMLIB_MOUNT_FLAG_READWRITE) {
+               ret = delete_staging_dir(wimfs_ctx);
+               if (ret != 0) {
+                       ERROR("Failed to delete the staging directory");
+                       if (status == 0)
+                               status = ret;
+               }
+       }
+       send_unmount_finished_msg(wimfs_ctx->daemon_to_unmount_mq, status);
+       return MSG_BREAK_LOOP;
 }
 
-/* Send a message to the filesystem daemon saying whether to commit or not. */
-static int send_unmount_request(struct wimfs_context *ctx, int unmount_flags)
+static int msg_daemon_info_handler(const void *_msg, void *_handler_ctx)
 {
-       int ret;
-       struct unmount_request msg;
+       const struct msg_daemon_info *msg = _msg;
+       struct unmount_msg_handler_context *handler_ctx = _handler_ctx;
 
-       msg.hdr.min_version    = WIMLIB_MAKEVERSION(1, 2, 0);
-       msg.hdr.cur_version    = WIMLIB_VERSION_CODE;
-       msg.hdr.data_size      = sizeof(msg.data);
-       msg.data.unmount_flags = unmount_flags;
+       DEBUG("Handling daemon info msg");
+       if (msg->hdr.msg_size < sizeof(*msg))
+               return WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       handler_ctx->daemon_pid = msg->daemon_pid;
+       handler_ctx->mount_flags = msg->mount_flags;
+       handler_ctx->hdr.timeout_seconds = 1;
+       DEBUG("pid of daemon is %d; mount flags were %#x",
+             handler_ctx->daemon_pid,
+             handler_ctx->mount_flags);
+       return 0;
+}
 
-       ret = mq_send(ctx->unmount_to_daemon_mq, (void*)&msg,
-                     sizeof(msg), 1);
-       if (ret == -1) {
-               ERROR_WITH_ERRNO("Failed to notify filesystem daemon whether "
-                                "we want to commit changes or not");
-               return WIMLIB_ERR_MQUEUE;
+static int msg_write_streams_progress_handler(const void *_msg,
+                                             void *_handler_ctx)
+{
+       const struct msg_write_streams_progress *msg = _msg;
+       struct unmount_msg_handler_context *handler_ctx = _handler_ctx;
+
+       if (msg->hdr.msg_size < sizeof(*msg))
+               return WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       if (handler_ctx->progress_func) {
+               handler_ctx->progress_func(WIMLIB_PROGRESS_MSG_WRITE_STREAMS,
+                                          &msg->info);
        }
        return 0;
 }
 
-static int receive_unmount_request(struct wimfs_context *ctx,
-                                  int *unmount_flags)
+static int msg_unmount_finished_handler(const void *_msg, void *_handler_ctx)
 {
-       void *mailbox;
-       long msgsize;
-       struct timeval now;
-       struct timespec timeout;
-       ssize_t bytes_received;
-       struct unmount_request *msg;
-       int ret;
+       const struct msg_unmount_finished *msg = _msg;
+       struct unmount_msg_handler_context *handler_ctx = _handler_ctx;
 
-       ret = get_mailbox(ctx->unmount_to_daemon_mq,
-                         sizeof(struct unmount_request),
-                         &msgsize, &mailbox);
-       if (ret != 0)
-               return ret;
+       DEBUG("Handling unmount finished message");
+       if (msg->hdr.msg_size < sizeof(*msg))
+               return WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       handler_ctx->status = msg->status;
+       DEBUG("status is %d", handler_ctx->status);
+       return MSG_BREAK_LOOP;
+}
 
-       /* Wait at most 3 seconds before giving up and discarding changes. */
-       gettimeofday(&now, NULL);
-       timeout.tv_sec = now.tv_sec + 3;
-       timeout.tv_nsec = now.tv_usec * 1000;
+static int unmount_timed_out_cb(void *_handler_ctx)
+{
+       struct unmount_msg_handler_context *handler_ctx = _handler_ctx;
 
-       bytes_received = mq_timedreceive(ctx->unmount_to_daemon_mq, mailbox,
-                                        msgsize, NULL, &timeout);
-       msg = mailbox;
-       if (bytes_received == -1) {
-               ERROR_WITH_ERRNO("mq_timedreceive()");
-               ERROR("Not committing.");
-               ret = WIMLIB_ERR_MQUEUE;
-       } else if (bytes_received < sizeof(struct unmount_message_header) ||
-                  bytes_received != sizeof(struct unmount_message_header) + msg->hdr.data_size) {
-               ERROR("Received invalid unmount request");
-               ret = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
-       } else if (WIMLIB_VERSION_CODE < msg->hdr.min_version) {
-               ERROR("Cannot understand the unmount request. "
-                     "Please upgrade wimlib to at least v%d.%d.%d",
-                     WIMLIB_GET_MAJOR_VERSION(msg->hdr.min_version),
-                     WIMLIB_GET_MINOR_VERSION(msg->hdr.min_version),
-                     WIMLIB_GET_PATCH_VERSION(msg->hdr.min_version));
-               ret = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       if (handler_ctx->daemon_pid == 0) {
+               goto out_crashed;
        } else {
-               *unmount_flags = msg->data.unmount_flags;
-               ret = 0;
+               kill(handler_ctx->daemon_pid, 0);
+               if (errno == ESRCH) {
+                       goto out_crashed;
+               } else {
+                       DEBUG("Filesystem daemon is still alive... "
+                             "Waiting another %d seconds\n",
+                             handler_ctx->hdr.timeout_seconds);
+                       return 0;
+               }
        }
-       FREE(mailbox);
-       return ret;
+out_crashed:
+       ERROR("The filesystem daemon has crashed!  Changes to the "
+             "WIM may not have been commited.");
+       return WIMLIB_ERR_FILESYSTEM_DAEMON_CRASHED;
 }
 
-static void send_unmount_response(struct wimfs_context *ctx, int status)
+static int daemon_timed_out_cb(void *_handler_ctx)
 {
-       struct unmount_response response;
-       response.hdr.min_version  = WIMLIB_MAKEVERSION(1, 2, 0);
-       response.hdr.cur_version  = WIMLIB_VERSION_CODE;
-       response.hdr.data_size    = sizeof(response.data);
-       response.data.status      = status;
-       response.data.mount_flags = ctx->mount_flags;
-       if (mq_send(ctx->daemon_to_unmount_mq, (void*)&response,
-                   sizeof(response), 1))
-               ERROR_WITH_ERRNO("Failed to send status to unmount process");
+       ERROR("Timed out waiting for unmount request! "
+             "Changes to the mounted WIM will not be committed.");
+       return WIMLIB_ERR_TIMEOUT;
 }
 
-/* Wait for a message from the filesytem daemon indicating whether  the
- * filesystem was unmounted successfully.  This may
- * take a long time if a big WIM file needs to be rewritten.
- *
- * Wait at most 600??? seconds before giving up and returning an error.  Either
- * it's a really big WIM file, or (more likely) the filesystem daemon has
- * crashed or failed for some reason.
- *
- * XXX come up with some method to determine if the filesystem daemon has really
- * crashed or not.
- *
- * XXX Idea: have mount daemon write its PID into the WIM file header?  No, this
- * wouldn't work because we know the directory but not the WIM file...
- */
-static int receive_unmount_response(struct wimfs_context *ctx)
+typedef int (*msg_handler_t)(const void *_msg, void *_handler_ctx);
+
+struct msg_handler_callbacks {
+       int (*timed_out)(void * _handler_ctx);
+       msg_handler_t msg_handlers[MSG_TYPE_MAX];
+};
+
+static const struct msg_handler_callbacks unmount_msg_handler_callbacks = {
+       .timed_out = unmount_timed_out_cb,
+       .msg_handlers = {
+               [MSG_TYPE_DAEMON_INFO] = msg_daemon_info_handler,
+               [MSG_TYPE_WRITE_STREAMS_PROGRESS] = msg_write_streams_progress_handler,
+               [MSG_TYPE_UNMOUNT_FINISHED] = msg_unmount_finished_handler,
+       },
+};
+
+static const struct msg_handler_callbacks daemon_msg_handler_callbacks = {
+       .timed_out = daemon_timed_out_cb,
+       .msg_handlers = {
+               [MSG_TYPE_UNMOUNT_REQUEST] = msg_unmount_request_handler,
+       },
+};
+
+static int receive_message(mqd_t mq,
+                          struct msg_handler_context_hdr *handler_ctx,
+                          const msg_handler_t msg_handlers[],
+                          long mailbox_size, void *mailbox)
 {
-       long msgsize;
-       void *mailbox;
        struct timeval now;
        struct timespec timeout;
        ssize_t bytes_received;
+       struct unmount_msg_hdr *hdr;
        int ret;
-       struct unmount_response *msg;
-
-       ret = get_mailbox(ctx->daemon_to_unmount_mq,
-                         sizeof(struct unmount_response), &msgsize, &mailbox);
-       if (ret != 0)
-               return ret;
-
-       DEBUG("Waiting for message telling us whether the unmount was "
-             "successful or not.");
 
        gettimeofday(&now, NULL);
-       timeout.tv_sec = now.tv_sec + 600;
+       timeout.tv_sec = now.tv_sec + handler_ctx->timeout_seconds;
        timeout.tv_nsec = now.tv_usec * 1000;
-       bytes_received = mq_timedreceive(ctx->daemon_to_unmount_mq,
-                                        mailbox, msgsize, NULL, &timeout);
-       msg = mailbox;
+
+       bytes_received = mq_timedreceive(mq, mailbox,
+                                        mailbox_size, NULL, &timeout);
+       hdr = mailbox;
        if (bytes_received == -1) {
                if (errno == ETIMEDOUT) {
-                       ERROR("Timed out- probably the filesystem daemon "
-                             "crashed and the WIM was not written "
-                             "successfully.");
                        ret = WIMLIB_ERR_TIMEOUT;
                } else {
-                       ERROR_WITH_ERRNO("mq_receive()");
+                       ERROR_WITH_ERRNO("mq_timedreceive()");
                        ret = WIMLIB_ERR_MQUEUE;
                }
-       } else if (bytes_received < sizeof(struct unmount_message_header) ||
-                  bytes_received != sizeof(struct unmount_message_header) + msg->hdr.data_size ||
-                  WIMLIB_VERSION_CODE < msg->hdr.min_version) {
-               ERROR("Received invalid unmount response");
+       } else if (bytes_received < sizeof(*hdr) ||
+                  bytes_received != hdr->msg_size) {
                ret = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
-       } else if (WIMLIB_VERSION_CODE < msg->hdr.min_version) {
-               ERROR("Cannot understand the unmount response. "
-                     "Please upgrade wimlib to at least v%d.%d.%d",
-                     WIMLIB_GET_MAJOR_VERSION(msg->hdr.min_version),
-                     WIMLIB_GET_MINOR_VERSION(msg->hdr.min_version),
-                     WIMLIB_GET_PATCH_VERSION(msg->hdr.min_version));
+       } else if (WIMLIB_VERSION_CODE < hdr->min_version) {
+               /*ERROR("Cannot understand the received message. "*/
+                     /*"Please upgrade wimlib to at least v%d.%d.%d",*/
+                     /*WIMLIB_GET_MAJOR_VERSION(hdr->min_version),*/
+                     /*WIMLIB_GET_MINOR_VERSION(hdr->min_version),*/
+                     /*WIMLIB_GET_PATCH_VERSION(hdr->min_version));*/
+               ret = MSG_VERSION_TOO_HIGH;
+       } else if (hdr->msg_type >= MSG_TYPE_MAX) {
+               ret = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       } else if (msg_handlers[hdr->msg_type] == NULL) {
                ret = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
        } else {
-               ret = msg->data.status;
-               if (ret) {
-                       ERROR("A failure status (%d) was returned "
-                             "by the filesystem daemon", ret);
+               ret = msg_handlers[hdr->msg_type](mailbox, handler_ctx);
+       }
+       return ret;
+}
+
+static int message_loop(mqd_t mq,
+                       const struct msg_handler_callbacks *callbacks,
+                       struct msg_handler_context_hdr *handler_ctx)
+{
+       static const size_t MAX_MSG_SIZE = 512;
+       long msgsize;
+       void *mailbox;
+       int ret;
+
+       DEBUG("Entering message loop");
+
+       ret = get_mailbox(mq, MAX_MSG_SIZE, &msgsize, &mailbox);
+       if (ret != 0)
+               return ret;
+       while (1) {
+               ret = receive_message(mq, handler_ctx,
+                                     callbacks->msg_handlers,
+                                     msgsize, mailbox);
+               if (ret == 0 || ret == MSG_VERSION_TOO_HIGH) {
+                       continue;
+               } else if (ret == MSG_BREAK_LOOP) {
+                       ret = 0;
+                       break;
+               } else if (ret == WIMLIB_ERR_TIMEOUT) {
+                       if (callbacks->timed_out)
+                               ret = callbacks->timed_out(handler_ctx);
+                       if (ret == 0)
+                               continue;
+                       else
+                               break;
+               } else {
+                       ERROR_WITH_ERRNO("Error communicating with "
+                                        "filesystem daemon");
+                       break;
                }
        }
        FREE(mailbox);
+       DEBUG("Exiting message loop");
        return ret;
 }
 
@@ -1175,46 +1410,53 @@ static int execute_fusermount(const char *dir)
        }
 
        /* Parent */
-       ret = wait(&status);
+       ret = waitpid(pid, &status, 0);
        if (ret == -1) {
                ERROR_WITH_ERRNO("Failed to wait for fusermount process to "
                                 "terminate");
                return WIMLIB_ERR_FUSERMOUNT;
        }
 
-       if (status != 0) {
-               if (status == WIMLIB_ERR_FUSERMOUNT)
-                       ERROR("Could not find the `fusermount' program");
-               else
-                       ERROR("fusermount exited with status %d", status);
+       if (!WIFEXITED(status)) {
+               ERROR("'fusermount' did not terminate normally!");
+               return WIMLIB_ERR_FUSERMOUNT;
+       }
 
-               /* Try again, but with the `umount' program.  This is required
-                * on other FUSE implementations such as FreeBSD's that do not
-                * have a `fusermount' program. */
+       status = WEXITSTATUS(status);
 
-               pid = fork();
-               if (pid == -1) {
-                       ERROR_WITH_ERRNO("Failed to fork()");
-                       return WIMLIB_ERR_FORK;
-               }
-               if (pid == 0) {
-                       /* Child */
-                       execlp("umount", "umount", dir, NULL);
-                       ERROR_WITH_ERRNO("Failed to execute `umount'");
-                       exit(WIMLIB_ERR_FUSERMOUNT);
-               }
+       if (status == 0)
+               return 0;
 
-               /* Parent */
-               ret = wait(&status);
-               if (ret == -1) {
-                       ERROR_WITH_ERRNO("Failed to wait for `umount' process to "
-                                        "terminate");
-                       return WIMLIB_ERR_FUSERMOUNT;
-               }
-               if (status != 0) {
-                       ERROR("`umount' exited with failure status");
-                       return WIMLIB_ERR_FUSERMOUNT;
-               }
+       if (status != WIMLIB_ERR_FUSERMOUNT)
+               return WIMLIB_ERR_FUSERMOUNT;
+
+       /* Try again, but with the `umount' program.  This is required on other
+        * FUSE implementations such as FreeBSD's that do not have a
+        * `fusermount' program. */
+       ERROR("Falling back to 'umount'.  Note: you may need to be "
+             "root for this to work");
+       pid = fork();
+       if (pid == -1) {
+               ERROR_WITH_ERRNO("Failed to fork()");
+               return WIMLIB_ERR_FORK;
+       }
+       if (pid == 0) {
+               /* Child */
+               execlp("umount", "umount", dir, NULL);
+               ERROR_WITH_ERRNO("Failed to execute `umount'");
+               exit(WIMLIB_ERR_FUSERMOUNT);
+       }
+
+       /* Parent */
+       ret = waitpid(pid, &status, 0);
+       if (ret == -1) {
+               ERROR_WITH_ERRNO("Failed to wait for `umount' process to "
+                                "terminate");
+               return WIMLIB_ERR_FUSERMOUNT;
+       }
+       if (status != 0) {
+               ERROR("`umount' did not successfully complete");
+               return WIMLIB_ERR_FUSERMOUNT;
        }
        return 0;
 }
@@ -1227,9 +1469,9 @@ static int wimfs_access(const char *path, int mask)
 
 static int wimfs_chmod(const char *path, mode_t mask)
 {
-       struct dentry *dentry;
+       struct wim_dentry *dentry;
        struct wimfs_context *ctx = wimfs_get_context();
-       struct inode *inode;
+       struct wim_inode *inode;
        struct stat stbuf;
        int ret;
 
@@ -1249,54 +1491,26 @@ static int wimfs_chmod(const char *path, mode_t mask)
 /* Called when the filesystem is unmounted. */
 static void wimfs_destroy(void *p)
 {
-       /* The `imagex unmount' command, which is running in a separate process
-        * and is executing the wimlib_unmount() function, will send this
-        * process a `struct unmount_request' through a message queue that
-        * specifies some unmount options, such as other to commit changes or
-        * not. */
-
-       int ret;
-       int unmount_flags;
-       struct wimfs_context *ctx;
-       int status;
-
-       ctx = wimfs_get_context();
-       if (open_message_queues(ctx, true))
-               return;
-
-       ret = receive_unmount_request(ctx, &unmount_flags);
-       status = ret;
-       if (ret == 0) {
-               if (ctx->mount_flags & WIMLIB_MOUNT_FLAG_READWRITE) {
-                       if (unmount_flags & WIMLIB_UNMOUNT_FLAG_COMMIT) {
-                               int write_flags = 0;
-                               if (unmount_flags & WIMLIB_UNMOUNT_FLAG_CHECK_INTEGRITY)
-                                       write_flags |= WIMLIB_WRITE_FLAG_CHECK_INTEGRITY;
-                               if (unmount_flags & WIMLIB_UNMOUNT_FLAG_REBUILD)
-                                       write_flags |= WIMLIB_WRITE_FLAG_REBUILD;
-                               if (unmount_flags & WIMLIB_UNMOUNT_FLAG_RECOMPRESS)
-                                       write_flags |= WIMLIB_WRITE_FLAG_RECOMPRESS;
-                               status = rebuild_wim(ctx, write_flags);
-                       }
-                       ret = delete_staging_dir(ctx);
-                       if (ret != 0) {
-                               ERROR("Failed to delete the staging directory");
-                               if (status == 0)
-                                       status = ret;
-                       }
-               } else {
-                       DEBUG("Read-only mount");
-               }
+       struct wimfs_context *wimfs_ctx = wimfs_get_context();
+       if (open_message_queues(wimfs_ctx, true) == 0) {
+               struct daemon_msg_handler_context handler_ctx = {
+                       .hdr = {
+                               .timeout_seconds = 5,
+                       },
+                       .wimfs_ctx = wimfs_ctx,
+               };
+               message_loop(wimfs_ctx->unmount_to_daemon_mq,
+                            &daemon_msg_handler_callbacks,
+                            &handler_ctx.hdr);
+               close_message_queues(wimfs_ctx);
        }
-       send_unmount_response(ctx, status);
-       close_message_queues(ctx);
 }
 
 #if 0
 static int wimfs_fallocate(const char *path, int mode,
                           off_t offset, off_t len, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        wimlib_assert(fd->staging_fd != -1);
        return fallocate(fd->staging_fd, mode, offset, len);
 }
@@ -1306,14 +1520,14 @@ static int wimfs_fallocate(const char *path, int mode,
 static int wimfs_fgetattr(const char *path, struct stat *stbuf,
                          struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        return inode_to_stbuf(fd->f_inode, fd->f_lte, stbuf);
 }
 
 static int wimfs_ftruncate(const char *path, off_t size,
                           struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        int ret = ftruncate(fd->staging_fd, size);
        if (ret != 0)
                return -errno;
@@ -1327,8 +1541,8 @@ static int wimfs_ftruncate(const char *path, off_t size,
  */
 static int wimfs_getattr(const char *path, struct stat *stbuf)
 {
-       struct dentry *dentry;
-       struct lookup_table_entry *lte;
+       struct wim_dentry *dentry;
+       struct wim_lookup_table_entry *lte;
        int ret;
        struct wimfs_context *ctx = wimfs_get_context();
 
@@ -1346,10 +1560,10 @@ static int wimfs_getxattr(const char *path, const char *name, char *value,
                          size_t size)
 {
        int ret;
-       struct inode *inode;
-       struct ads_entry *ads_entry;
+       struct wim_inode *inode;
+       struct wim_ads_entry *ads_entry;
        size_t res_size;
-       struct lookup_table_entry *lte;
+       struct wim_lookup_table_entry *lte;
        struct wimfs_context *ctx = wimfs_get_context();
 
        if (!(ctx->mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_XATTR))
@@ -1361,7 +1575,7 @@ static int wimfs_getxattr(const char *path, const char *name, char *value,
 
        inode = wim_pathname_to_inode(ctx->wim, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
        ads_entry = inode_get_ads_entry(inode, name, NULL);
        if (!ads_entry)
@@ -1388,26 +1602,24 @@ static int wimfs_getxattr(const char *path, const char *name, char *value,
 /* Create a hard link */
 static int wimfs_link(const char *to, const char *from)
 {
-       struct dentry *from_dentry, *from_dentry_parent;
+       struct wim_dentry *from_dentry, *from_dentry_parent;
        const char *link_name;
-       struct inode *inode;
-       struct lookup_table_entry *lte;
+       struct wim_inode *inode;
+       struct wim_lookup_table_entry *lte;
        WIMStruct *w = wimfs_get_WIMStruct();
        u16 i;
 
        inode = wim_pathname_to_inode(w, to);
        if (!inode)
-               return -ENOENT;
-
-       if (inode->attributes & FILE_ATTRIBUTE_REPARSE_POINT)
-               return -EEXIST;
+               return -errno;
 
-       if (inode->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;
 
@@ -1416,13 +1628,13 @@ static int wimfs_link(const char *to, const char *from)
                return -EEXIST;
        from_dentry = new_dentry(link_name);
        if (!from_dentry)
-               return -ENOMEM;
+               return -errno;
 
        inode_add_dentry(from_dentry, inode);
        from_dentry->d_inode = inode;
-       inode->link_count++;
+       inode->i_nlink++;
 
-       for (i = 0; i <= inode->num_ads; i++) {
+       for (i = 0; i <= inode->i_num_ads; i++) {
                lte = inode_stream_lte_resolved(inode, i);
                if (lte)
                        lte->refcnt++;
@@ -1435,7 +1647,7 @@ static int wimfs_link(const char *to, const char *from)
 static int wimfs_listxattr(const char *path, char *list, size_t size)
 {
        size_t needed_size;
-       struct inode *inode;
+       struct wim_inode *inode;
        struct wimfs_context *ctx = wimfs_get_context();
        u16 i;
        char *p;
@@ -1447,21 +1659,21 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
 
        inode = wim_pathname_to_inode(ctx->wim, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
        if (size == 0) {
                needed_size = 0;
-               for (i = 0; i < inode->num_ads; i++)
-                       needed_size += inode->ads_entries[i].stream_name_utf8_len + 6;
+               for (i = 0; i < inode->i_num_ads; i++)
+                       needed_size += inode->i_ads_entries[i].stream_name_utf8_len + 6;
                return needed_size;
        } else {
                p = list;
-               for (i = 0; i < inode->num_ads; i++) {
-                       needed_size = inode->ads_entries[i].stream_name_utf8_len + 6;
+               for (i = 0; i < inode->i_num_ads; i++) {
+                       needed_size = inode->i_ads_entries[i].stream_name_utf8_len + 6;
                        if (needed_size > size)
                                return -ERANGE;
                        p += sprintf(p, "user.%s",
-                                    inode->ads_entries[i].stream_name_utf8) + 1;
+                                    inode->i_ads_entries[i].stream_name_utf8) + 1;
                        size -= needed_size;
                }
                return p - list;
@@ -1474,12 +1686,12 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
  * @mode is currently ignored.  */
 static int wimfs_mkdir(const char *path, mode_t mode)
 {
-       struct dentry *dentry;
+       struct wim_dentry *dentry;
        int ret;
 
        ret = create_dentry(wimfs_get_context(), path, &dentry);
        if (ret == 0)
-               dentry->d_inode->attributes = FILE_ATTRIBUTE_DIRECTORY;
+               dentry->d_inode->i_attributes = FILE_ATTRIBUTE_DIRECTORY;
        return ret;
 }
 
@@ -1492,8 +1704,8 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
        if ((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 inode *inode;
+               struct wim_ads_entry *new_entry;
+               struct wim_inode *inode;
 
                char *p = (char*)stream_name - 1;
                wimlib_assert(*p == ':');
@@ -1501,8 +1713,8 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
 
                inode = wim_pathname_to_inode(ctx->wim, path);
                if (!inode)
-                       return -ENOENT;
-               if (inode->attributes &
+                       return -errno;
+               if (inode->i_attributes &
                    (FILE_ATTRIBUTE_REPARSE_POINT | FILE_ATTRIBUTE_DIRECTORY))
                        return -ENOENT;
                if (inode_get_ads_entry(inode, stream_name, NULL))
@@ -1512,13 +1724,13 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
                        return -ENOMEM;
                return 0;
        } else {
-               struct dentry *dentry;
+               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->attributes = FILE_ATTRIBUTE_NORMAL;
+                       dentry->d_inode->i_attributes = FILE_ATTRIBUTE_NORMAL;
                return ret;
        }
 }
@@ -1527,11 +1739,11 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
 /* Open a file.  */
 static int wimfs_open(const char *path, struct fuse_file_info *fi)
 {
-       struct dentry *dentry;
-       struct lookup_table_entry *lte;
+       struct wim_dentry *dentry;
+       struct wim_lookup_table_entry *lte;
        int ret;
-       struct wimlib_fd *fd;
-       struct inode *inode;
+       struct wimfs_fd *fd;
+       struct wim_inode *inode;
        u16 stream_idx;
        u32 stream_id;
        struct wimfs_context *ctx = wimfs_get_context();
@@ -1546,7 +1758,7 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
        if (stream_idx == 0)
                stream_id = 0;
        else
-               stream_id = inode->ads_entries[stream_idx - 1].stream_id;
+               stream_id = inode->i_ads_entries[stream_idx - 1].stream_id;
 
        /* The file resource may be in the staging directory (read-write mounts
         * only) or in the WIM.  If it's in the staging directory, we need to
@@ -1564,8 +1776,8 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
                        return ret;
        }
 
-       ret = alloc_wimlib_fd(inode, stream_id, lte, &fd,
-                             wimfs_ctx_readonly(ctx));
+       ret = alloc_wimfs_fd(inode, stream_id, lte, &fd,
+                            wimfs_ctx_readonly(ctx));
        if (ret != 0)
                return ret;
 
@@ -1573,7 +1785,7 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
                fd->staging_fd = open(lte->staging_file_name, fi->flags);
                if (fd->staging_fd == -1) {
                        int errno_save = errno;
-                       close_wimlib_fd(fd);
+                       close_wimfs_fd(fd);
                        return -errno_save;
                }
        }
@@ -1584,18 +1796,18 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
 /* Opens a directory. */
 static int wimfs_opendir(const char *path, struct fuse_file_info *fi)
 {
-       struct inode *inode;
+       struct wim_inode *inode;
        int ret;
-       struct wimlib_fd *fd = NULL;
+       struct wimfs_fd *fd = NULL;
        struct wimfs_context *ctx = wimfs_get_context();
        WIMStruct *w = ctx->wim;
 
        inode = wim_pathname_to_inode(w, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
        if (!inode_is_directory(inode))
                return -ENOTDIR;
-       ret = alloc_wimlib_fd(inode, 0, NULL, &fd, wimfs_ctx_readonly(ctx));
+       ret = alloc_wimfs_fd(inode, 0, NULL, &fd, wimfs_ctx_readonly(ctx));
        fi->fh = (uintptr_t)fd;
        return ret;
 }
@@ -1607,7 +1819,7 @@ static int wimfs_opendir(const char *path, struct fuse_file_info *fi)
 static int wimfs_read(const char *path, char *buf, size_t size,
                      off_t offset, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        ssize_t ret;
 
        if (!fd)
@@ -1622,7 +1834,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;
@@ -1649,7 +1861,7 @@ struct fill_params {
        fuse_fill_dir_t filler;
 };
 
-static int dentry_fuse_fill(struct dentry *dentry, void *arg)
+static int dentry_fuse_fill(struct wim_dentry *dentry, void *arg)
 {
        struct fill_params *fill_params = arg;
        return fill_params->filler(fill_params->buf, dentry->file_name_utf8,
@@ -1661,8 +1873,8 @@ static int dentry_fuse_fill(struct dentry *dentry, void *arg)
 static int wimfs_readdir(const char *path, void *buf, fuse_fill_dir_t filler,
                         off_t offset, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
-       struct inode *inode;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
+       struct wim_inode *inode;
 
        if (!fd)
                return -EBADF;
@@ -1677,7 +1889,7 @@ static int wimfs_readdir(const char *path, void *buf, fuse_fill_dir_t filler,
        filler(buf, ".", NULL, 0);
        filler(buf, "..", NULL, 0);
 
-       return for_dentry_in_rbtree(inode->children.rb_node,
+       return for_dentry_in_rbtree(inode->i_children.rb_node,
                                    dentry_fuse_fill, &fill_params);
 }
 
@@ -1685,10 +1897,10 @@ static int wimfs_readdir(const char *path, void *buf, fuse_fill_dir_t filler,
 static int wimfs_readlink(const char *path, char *buf, size_t buf_len)
 {
        struct wimfs_context *ctx = wimfs_get_context();
-       struct inode *inode = wim_pathname_to_inode(ctx->wim, path);
+       struct wim_inode *inode = wim_pathname_to_inode(ctx->wim, path);
        int ret;
        if (!inode)
-               return -ENOENT;
+               return -errno;
        if (!inode_is_symlink(inode))
                return -EINVAL;
 
@@ -1702,23 +1914,23 @@ static int wimfs_readlink(const char *path, char *buf, size_t buf_len)
 /* Close a file. */
 static int wimfs_release(const char *path, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
-       return close_wimlib_fd(fd);
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
+       return close_wimfs_fd(fd);
 }
 
 /* Close a directory */
 static int wimfs_releasedir(const char *path, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
-       return close_wimlib_fd(fd);
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
+       return close_wimfs_fd(fd);
 }
 
 #ifdef ENABLE_XATTR
 /* Remove an alternate data stream through the XATTR interface */
 static int wimfs_removexattr(const char *path, const char *name)
 {
-       struct inode *inode;
-       struct ads_entry *ads_entry;
+       struct wim_inode *inode;
+       struct wim_ads_entry *ads_entry;
        u16 ads_idx;
        struct wimfs_context *ctx = wimfs_get_context();
 
@@ -1731,7 +1943,7 @@ static int wimfs_removexattr(const char *path, const char *name)
 
        inode = wim_pathname_to_inode(ctx->wim, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
        ads_entry = inode_get_ads_entry(inode, name, &ads_idx);
        if (!ads_entry)
@@ -1744,11 +1956,9 @@ static int wimfs_removexattr(const char *path, const char *name)
 /* Renames a file or directory.  See rename (3) */
 static int wimfs_rename(const char *from, const char *to)
 {
-       struct dentry *src;
-       struct dentry *dst;
-       struct dentry *parent_of_dst;
-       char *file_name_utf16 = NULL, *file_name_utf8 = NULL;
-       u16 file_name_utf16_len, file_name_utf8_len;
+       struct wim_dentry *src;
+       struct wim_dentry *dst;
+       struct wim_dentry *parent_of_dst;
        WIMStruct *w = wimfs_get_WIMStruct();
        int ret;
 
@@ -1757,16 +1967,10 @@ static int wimfs_rename(const char *from, const char *to)
 
        src = get_dentry(w, from);
        if (!src)
-               return -ENOENT;
+               return -errno;
 
        dst = get_dentry(w, to);
 
-       ret = get_names(&file_name_utf16, &file_name_utf8,
-                       &file_name_utf16_len, &file_name_utf8_len,
-                       path_basename(to));
-       if (ret != 0)
-               return -ENOMEM;
-
        if (dst) {
                /* Destination file exists */
 
@@ -1786,24 +1990,21 @@ static int wimfs_rename(const char *from, const char *to)
                                return -ENOTEMPTY;
                }
                parent_of_dst = dst->parent;
-               remove_dentry(dst, w->lookup_table);
        } else {
                /* Destination does not exist */
                parent_of_dst = get_parent_dentry(w, to);
                if (!parent_of_dst)
-                       return -ENOENT;
+                       return -errno;
 
                if (!dentry_is_directory(parent_of_dst))
                        return -ENOTDIR;
        }
 
-       FREE(src->file_name);
-       FREE(src->file_name_utf8);
-       src->file_name          = file_name_utf16;
-       src->file_name_utf8     = file_name_utf8;
-       src->file_name_len      = file_name_utf16_len;
-       src->file_name_utf8_len = file_name_utf8_len;
-
+       ret = set_dentry_name(src, path_basename(to));
+       if (ret != 0)
+               return -ENOMEM;
+       if (dst)
+               remove_dentry(dst, w->lookup_table);
        unlink_dentry(src);
        dentry_add_child(parent_of_dst, src);
        return 0;
@@ -1812,14 +2013,17 @@ static int wimfs_rename(const char *from, const char *to)
 /* Remove a directory */
 static int wimfs_rmdir(const char *path)
 {
-       struct dentry *dentry;
+       struct wim_dentry *parent, *dentry;
        WIMStruct *w = wimfs_get_WIMStruct();
 
        dentry = get_dentry(w, path);
        if (!dentry)
-               return -ENOENT;
+               return -errno;
+
+       if (!dentry_is_directory(dentry))
+               return -ENOTDIR;
 
-       if (!dentry_is_empty_directory(dentry))
+       if (dentry_has_children(dentry))
                return -ENOTEMPTY;
 
        remove_dentry(dentry, w->lookup_table);
@@ -1831,11 +2035,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 lookup_table_entry *existing_lte;
-       struct lookup_table_entry *lte;
-       struct inode *inode;
+       struct wim_ads_entry *existing_ads_entry;
+       struct wim_ads_entry *new_ads_entry;
+       struct wim_lookup_table_entry *existing_lte;
+       struct wim_lookup_table_entry *lte;
+       struct wim_inode *inode;
        u8 value_hash[SHA1_HASH_SIZE];
        u16 ads_idx;
        struct wimfs_context *ctx = wimfs_get_context();
@@ -1849,7 +2053,7 @@ static int wimfs_setxattr(const char *path, const char *name,
 
        inode = wim_pathname_to_inode(ctx->wim, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
        existing_ads_entry = inode_get_ads_entry(inode, name, &ads_idx);
        if (existing_ads_entry) {
@@ -1898,13 +2102,13 @@ static int wimfs_setxattr(const char *path, const char *name,
 static int wimfs_symlink(const char *to, const char *from)
 {
        struct wimfs_context *ctx = wimfs_get_context();
-       struct dentry *dentry;
+       struct wim_dentry *dentry;
        int ret;
 
        ret = create_dentry(ctx, from, &dentry);
        if (ret == 0) {
-               dentry->d_inode->attributes = FILE_ATTRIBUTE_REPARSE_POINT;
-               dentry->d_inode->reparse_tag = WIM_IO_REPARSE_TAG_SYMLINK;
+               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))
                {
@@ -1920,12 +2124,12 @@ static int wimfs_symlink(const char *to, const char *from)
 /* Reduce the size of a file */
 static int wimfs_truncate(const char *path, off_t size)
 {
-       struct dentry *dentry;
-       struct lookup_table_entry *lte;
+       struct wim_dentry *dentry;
+       struct wim_lookup_table_entry *lte;
        int ret;
        u16 stream_idx;
        u32 stream_id;
-       struct inode *inode;
+       struct wim_inode *inode;
        struct wimfs_context *ctx = wimfs_get_context();
 
        ret = lookup_resource(ctx->wim, path, get_lookup_flags(ctx),
@@ -1941,7 +2145,7 @@ static int wimfs_truncate(const char *path, off_t size)
        if (stream_idx == 0)
                stream_id = 0;
        else
-               stream_id = inode->ads_entries[stream_idx - 1].stream_id;
+               stream_id = inode->i_ads_entries[stream_idx - 1].stream_id;
 
        if (lte->resource_location == RESOURCE_IN_STAGING_FILE) {
                ret = truncate(lte->staging_file_name, size);
@@ -1959,8 +2163,8 @@ static int wimfs_truncate(const char *path, off_t size)
 /* Unlink a non-directory or alternate data stream */
 static int wimfs_unlink(const char *path)
 {
-       struct dentry *dentry;
-       struct lookup_table_entry *lte;
+       struct wim_dentry *dentry;
+       struct wim_lookup_table_entry *lte;
        int ret;
        u16 stream_idx;
        struct wimfs_context *ctx = wimfs_get_context();
@@ -1987,43 +2191,43 @@ static int wimfs_unlink(const char *path)
  */
 static int wimfs_utimens(const char *path, const struct timespec tv[2])
 {
-       struct dentry *dentry;
-       struct inode *inode;
+       struct wim_dentry *dentry;
+       struct wim_inode *inode;
        WIMStruct *w = wimfs_get_WIMStruct();
 
        dentry = get_dentry(w, path);
        if (!dentry)
-               return -ENOENT;
+               return -errno;
        inode = dentry->d_inode;
 
        if (tv[0].tv_nsec != UTIME_OMIT) {
                if (tv[0].tv_nsec == UTIME_NOW)
-                       inode->last_access_time = get_wim_timestamp();
+                       inode->i_last_access_time = get_wim_timestamp();
                else
-                       inode->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->last_write_time = get_wim_timestamp();
+                       inode->i_last_write_time = get_wim_timestamp();
                else
-                       inode->last_write_time = timespec_to_wim_timestamp(&tv[1]);
+                       inode->i_last_write_time = timespec_to_wim_timestamp(&tv[1]);
        }
        return 0;
 }
 #else
 static int wimfs_utime(const char *path, struct utimbuf *times)
 {
-       struct dentry *dentry;
-       struct inode *inode;
+       struct wim_dentry *dentry;
+       struct wim_inode *inode;
        WIMStruct *w = wimfs_get_WIMStruct();
 
        dentry = get_dentry(w, path);
        if (!dentry)
-               return -ENOENT;
+               return -errno;
        inode = dentry->d_inode;
 
-       inode->last_write_time = unix_timestamp_to_wim(times->modtime);
-       inode->last_access_time = unix_timestamp_to_wim(times->actime);
+       inode->i_last_write_time = unix_timestamp_to_wim(times->modtime);
+       inode->i_last_access_time = unix_timestamp_to_wim(times->actime);
        return 0;
 }
 #endif
@@ -2034,7 +2238,7 @@ static int wimfs_utime(const char *path, struct utimbuf *times)
 static int wimfs_write(const char *path, const char *buf, size_t size,
                       off_t offset, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        int ret;
        u64 now;
 
@@ -2056,8 +2260,8 @@ static int wimfs_write(const char *path, const char *buf, size_t size,
                return -errno;
 
        now = get_wim_timestamp();
-       fd->f_inode->last_write_time = now;
-       fd->f_inode->last_access_time = now;
+       fd->f_inode->i_last_write_time = now;
+       fd->f_inode->i_last_access_time = now;
        return ret;
 }
 
@@ -2105,6 +2309,9 @@ static struct fuse_operations wimfs_operations = {
 #endif
        .write       = wimfs_write,
 
+       /* wimfs keeps file descriptor structures (struct wimfs_fd), so there is
+        * no need to have the file path provided on operations such as read()
+        * where only the file descriptor is needed. */
 #if FUSE_MAJOR_VERSION > 2 || (FUSE_MAJOR_VERSION == 2 && FUSE_MINOR_VERSION >= 8)
        .flag_nullpath_ok = 1,
 #endif
@@ -2125,11 +2332,11 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        char *argv[16];
        int ret;
        char *dir_copy;
-       struct lookup_table *joined_tab, *wim_tab_save;
-       struct image_metadata *imd;
+       struct wim_lookup_table *joined_tab, *wim_tab_save;
+       struct wim_image_metadata *imd;
        struct wimfs_context ctx;
        struct hlist_node *cur_node;
-       struct inode *inode;
+       struct wim_inode *inode;
 
        DEBUG("Mount: wim = %p, image = %d, dir = %s, flags = %d, ",
              wim, image, dir, mount_flags);
@@ -2177,6 +2384,9 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
                goto out;
        }
 
+       if (imd->inode_list.first) /* May be unneeded? */
+               imd->inode_list.first->pprev = &imd->inode_list.first;
+
        if (imd->modified) {
                ERROR("Cannot mount image that was added "
                      "with wimlib_add_image()");
@@ -2190,6 +2400,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
                        goto out;
        }
 
+       /* Use default stream interface if one was not specified */
        if (!(mount_flags & (WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_NONE |
                       WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_XATTR |
                       WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_WINDOWS)))
@@ -2228,11 +2439,8 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
                argv[argc++] = "-d";
 
        /*
-        * We provide the use_ino option because we are going to assign inode
-        * numbers oursides.  The inodes will be given unique numbers in the
-        * assign_inode_numbers() function, and the static variable @next_ino is
-        * set to the next available inode number.
-        */
+        * We provide the use_ino option to the FUSE mount because we are going
+        * to assign inode numbers ourselves. */
        char optstring[256] =
                "use_ino"
                ",subtype=wimfs"
@@ -2276,13 +2484,11 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        /* Resolve the lookup table entries for every inode in the image, and
         * assign inode numbers */
        DEBUG("Resolving lookup table entries and assigning inode numbers");
-
        ctx.next_ino = 1;
-       hlist_for_each_entry(inode, cur_node, &imd->inode_list, hlist) {
+       hlist_for_each_entry(inode, cur_node, &imd->inode_list, i_hlist) {
                inode_resolve_ltes(inode, wim->lookup_table);
-               inode->ino = ctx.next_ino++;
+               inode->i_ino = ctx.next_ino++;
        }
-       /*ctx.next_ino = assign_inode_numbers(&imd->inode_list);*/
        DEBUG("(next_ino = %"PRIu64")", ctx.next_ino);
 
        DEBUG("Calling fuse_main()");
@@ -2290,6 +2496,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        ret = fuse_main(argc, argv, &wimfs_operations, &ctx);
 
        DEBUG("Returned from fuse_main() (ret = %d)", ret);
+
        if (ret)
                ret = WIMLIB_ERR_FUSE;
 out_free_dir_copy:
@@ -2314,30 +2521,45 @@ WIMLIBAPI int wimlib_unmount_image(const char *dir, int unmount_flags,
                                   wimlib_progress_func_t progress_func)
 {
        int ret;
-       struct wimfs_context ctx;
+       struct wimfs_context wimfs_ctx;
 
-       init_wimfs_context(&ctx);
+       init_wimfs_context(&wimfs_ctx);
 
-       ret = set_message_queue_names(&ctx, dir);
+       ret = set_message_queue_names(&wimfs_ctx, dir);
        if (ret != 0)
                goto out;
 
-       ret = open_message_queues(&ctx, false);
+       ret = open_message_queues(&wimfs_ctx, false);
        if (ret != 0)
                goto out_free_message_queue_names;
 
-       ret = send_unmount_request(&ctx, unmount_flags);
+       ret = send_unmount_request_msg(wimfs_ctx.unmount_to_daemon_mq,
+                                      unmount_flags,
+                                      progress_func != NULL);
        if (ret != 0)
                goto out_close_message_queues;
 
        ret = execute_fusermount(dir);
        if (ret != 0)
                goto out_close_message_queues;
-       ret = receive_unmount_response(&ctx);
+
+       struct unmount_msg_handler_context handler_ctx = {
+               .hdr = {
+                       .timeout_seconds = 5,
+               },
+               .daemon_pid = 0,
+               .progress_func = progress_func,
+       };
+
+       ret = message_loop(wimfs_ctx.daemon_to_unmount_mq,
+                          &unmount_msg_handler_callbacks,
+                          &handler_ctx.hdr);
+       if (ret == 0)
+               ret = handler_ctx.status;
 out_close_message_queues:
-       close_message_queues(&ctx);
+       close_message_queues(&wimfs_ctx);
 out_free_message_queue_names:
-       free_message_queue_names(&ctx);
+       free_message_queue_names(&wimfs_ctx);
 out:
        return ret;
 }