X-Git-Url: https://wimlib.net/git/?a=blobdiff_plain;f=src%2Fupdate_image.c;h=317cca8a51d6c10cd99321cf33a78227f298b913;hb=4d9794d1f1a6044c261faae0968af1b2c0480f4b;hp=7247b1c06b8fb61064974f21c27e0baa670d1964;hpb=12d47b06960d51e4f254ebbc882b05445c992f1d;p=wimlib

diff --git a/src/update_image.c b/src/update_image.c
index 7247b1c0..317cca8a 100644
--- a/src/update_image.c
+++ b/src/update_image.c
@@ -21,16 +21,22 @@
  * along with wimlib; if not, see http://www.gnu.org/licenses/.
  */
 
-#include "wimlib_internal.h"
-#include "dentry.h"
-#include "lookup_table.h"
-#include "security.h"
-#include "xml.h"
-#include <errno.h>
+#ifdef HAVE_CONFIG_H
+#  include "config.h"
+#endif
 
-#ifdef __WIN32__
-#  include "win32.h"
+#include "wimlib/capture.h"
+#include "wimlib/dentry.h"
+#include "wimlib/error.h"
+#include "wimlib/lookup_table.h"
+#include "wimlib/metadata.h"
+#ifdef WITH_NTFS_3G
+#  include "wimlib/ntfs_3g.h" /* for do_ntfs_umount() */
 #endif
+#include "wimlib/paths.h"
+#include "wimlib/xml.h"
+
+#include <errno.h>
 
 /* Overlays @branch onto @target, both of which must be directories. */
 static int
@@ -48,6 +54,7 @@ do_overlay(struct wim_dentry *target, struct wim_dentry *branch)
 	}
 
 	rb_root = &branch->d_inode->i_children;
+	LIST_HEAD(moved_children);
 	while (rb_root->rb_node) { /* While @branch has children... */
 		struct wim_dentry *child = rbnode_dentry(rb_root->rb_node);
 		struct wim_dentry *existing;
@@ -61,12 +68,17 @@ do_overlay(struct wim_dentry *target, struct wim_dentry *branch)
 			int ret;
 			ret = do_overlay(existing, child);
 			if (ret) {
-				/* Overlay failed.  Revert the change to avoid
-				 * leaking the directory tree rooted at @child.
-				 * */
+				/* Overlay failed.  Revert the changes. */
 				dentry_add_child(branch, child);
+				list_for_each_entry(child, &moved_children, tmp_list)
+				{
+					unlink_dentry(child);
+					dentry_add_child(branch, child);
+				}
 				return ret;
 			}
+		} else {
+			list_add(&child->tmp_list, &moved_children);
 		}
 	}
 	free_dentry(branch);
@@ -162,8 +174,6 @@ execute_add_command(WIMStruct *wim,
 	int add_flags;
 	tchar *fs_source_path;
 	tchar *wim_target_path;
-	struct wim_inode_table inode_table;
-	struct sd_set sd_set;
 	struct wim_image_metadata *imd;
 	struct list_head unhashed_streams;
 	struct add_image_params params;
@@ -211,19 +221,17 @@ execute_add_command(WIMStruct *wim,
 		extra_arg = NULL;
 	}
 
-	ret = init_inode_table(&inode_table, 9001);
+	ret = init_inode_table(&params.inode_table, 9001);
 	if (ret)
 		goto out;
 
-	ret = init_sd_set(&sd_set, imd->security_data);
+	ret = init_sd_set(&params.sd_set, imd->security_data);
 	if (ret)
 		goto out_destroy_inode_table;
 
 	INIT_LIST_HEAD(&unhashed_streams);
 	wim->lookup_table->unhashed_streams = &unhashed_streams;
 	params.lookup_table = wim->lookup_table;
-	params.inode_table = &inode_table;
-	params.sd_set = &sd_set;
 	params.config = config;
 	params.add_flags = add_flags;
 	params.progress_func = progress_func;
@@ -235,8 +243,10 @@ execute_add_command(WIMStruct *wim,
 		progress.scan.wim_target_path = wim_target_path;
 		progress_func(WIMLIB_PROGRESS_MSG_SCAN_BEGIN, &progress);
 	}
-	config->_prefix = fs_source_path;
-	config->_prefix_num_tchars = tstrlen(fs_source_path);
+	if (config) {
+		config->_prefix = fs_source_path;
+		config->_prefix_num_tchars = tstrlen(fs_source_path);
+	}
 
 	if (wim_target_path[0] == T('\0'))
 		add_flags |= WIMLIB_ADD_FLAG_ROOT;
@@ -266,7 +276,7 @@ execute_add_command(WIMStruct *wim,
 #ifdef WITH_NTFS_3G
 	imd->ntfs_vol = ntfs_vol;
 #endif
-	inode_table_prepare_inode_list(&inode_table, &imd->inode_list);
+	inode_table_prepare_inode_list(&params.inode_table, &imd->inode_list);
 	ret = 0;
 	rollback_sd = false;
 	if (add_flags & WIMLIB_ADD_FLAG_RPFIX)
@@ -279,9 +289,9 @@ out_ntfs_umount:
 #endif
 	free_dentry_tree(branch, wim->lookup_table);
 out_destroy_sd_set:
-	destroy_sd_set(&sd_set, rollback_sd);
+	destroy_sd_set(&params.sd_set, rollback_sd);
 out_destroy_inode_table:
-	destroy_inode_table(&inode_table);
+	destroy_inode_table(&params.inode_table);
 out:
 	return ret;
 }
@@ -328,6 +338,14 @@ execute_delete_command(WIMStruct *wim,
 	return 0;
 }
 
+static int
+free_dentry_full_path(struct wim_dentry *dentry, void *_ignore)
+{
+	FREE(dentry->_full_path);
+	dentry->_full_path = NULL;
+	return 0;
+}
+
 /*
  * Rename a file or directory in the WIM.
  *
@@ -388,6 +406,8 @@ rename_wim_path(WIMStruct *wim, const tchar *from, const tchar *to)
 	}
 	unlink_dentry(src);
 	dentry_add_child(parent_of_dst, src);
+	if (src->_full_path)
+		for_dentry_in_tree(src, free_dentry_full_path, NULL);
 	return 0;
 }
 
@@ -442,6 +462,7 @@ update_op_to_str(int op)
 		return T("rename");
 	default:
 		wimlib_assert(0);
+		return NULL;
 	}
 }
 
@@ -577,7 +598,7 @@ check_update_commands(struct wimlib_update_command *cmds, size_t num_cmds,
 }
 
 
-extern void
+static void
 free_update_commands(struct wimlib_update_command *cmds, size_t num_cmds)
 {
 	if (cmds) {