X-Git-Url: https://wimlib.net/git/?a=blobdiff_plain;f=src%2Fjoin.c;h=d52bd0f242f1d4cab3d82588d3ba571e37243c05;hb=61ac5994e9fe3666d1c057dafca8c79406c59077;hp=2157697d7eb556e5758a2662b1db422e81d38787;hpb=0e8ba68f2bd5e3f844611b93b6335e0a24f5c327;p=wimlib

diff --git a/src/join.c b/src/join.c
index 2157697d..d52bd0f2 100644
--- a/src/join.c
+++ b/src/join.c
@@ -28,11 +28,119 @@
 #endif
 
 #include "wimlib.h"
+#include "wimlib/error.h"
 #include "wimlib/types.h"
-#include "wimlib/swm.h"
 #include "wimlib/util.h"
 #include "wimlib/wim.h"
 
+/*
+ * verify_swm_set: - Sanity checks to make sure a set of WIMs correctly
+ *		     correspond to a spanned set.
+ *
+ * @wim:
+ * 	Part 1 of the set.
+ *
+ * @additional_swms:
+ * 	All parts of the set other than part 1.
+ *
+ * @num_additional_swms:
+ * 	Number of WIMStructs in @additional_swms.  Or, the total number of parts
+ * 	in the set minus 1.
+ *
+ * @return:
+ * 	0 on success; WIMLIB_ERR_SPLIT_INVALID if the set is not valid.
+ */
+static int
+verify_swm_set(WIMStruct *wim, WIMStruct **additional_swms,
+	       unsigned num_additional_swms)
+{
+	unsigned total_parts = wim->hdr.total_parts;
+	int ctype;
+	const u8 *guid;
+
+	if (total_parts != num_additional_swms + 1) {
+		ERROR("`%"TS"' says there are %u parts in the spanned set, "
+		      "but %"TS"%u part%"TS" provided",
+		      wim->filename, total_parts,
+		      (num_additional_swms + 1 < total_parts) ? T("only ") : T(""),
+		      num_additional_swms + 1,
+		      (num_additional_swms) ? T("s were") : T(" was"));
+		return WIMLIB_ERR_SPLIT_INVALID;
+	}
+	if (wim->hdr.part_number != 1) {
+		ERROR("WIM `%"TS"' is not the first part of the split WIM.",
+		      wim->filename);
+		return WIMLIB_ERR_SPLIT_INVALID;
+	}
+	for (unsigned i = 0; i < num_additional_swms; i++) {
+		if (additional_swms[i]->hdr.total_parts != total_parts) {
+			ERROR("WIM `%"TS"' says there are %u parts in the "
+			      "spanned set, but %u parts were provided",
+			      additional_swms[i]->filename,
+			      additional_swms[i]->hdr.total_parts,
+			      total_parts);
+			return WIMLIB_ERR_SPLIT_INVALID;
+		}
+	}
+
+	/* keep track of ctype and guid just to make sure they are the same for
+	 * all the WIMs. */
+	ctype = wim->compression_type;
+	guid = wim->hdr.guid;
+
+	{
+		/* parts_to_swms is not allocated at function scope because it
+		 * should only be allocated after num_additional_swms was
+		 * checked to be the same as wim->hdr.total_parts.  Otherwise, it
+		 * could be unexpectedly high and cause a stack overflow. */
+		WIMStruct *parts_to_swms[num_additional_swms];
+		ZERO_ARRAY(parts_to_swms);
+		for (unsigned i = 0; i < num_additional_swms; i++) {
+
+			WIMStruct *swm = additional_swms[i];
+
+			if (swm->compression_type != ctype) {
+				ERROR("The split WIMs do not all have the same "
+				      "compression type");
+				return WIMLIB_ERR_SPLIT_INVALID;
+			}
+			if (memcmp(guid, swm->hdr.guid, WIM_GID_LEN) != 0) {
+				ERROR("The split WIMs do not all have the same "
+				      "GUID");
+				return WIMLIB_ERR_SPLIT_INVALID;
+			}
+			if (swm->hdr.part_number == 1) {
+				ERROR("WIMs `%"TS"' and `%"TS"' both are marked "
+				      "as the first WIM in the spanned set",
+				      wim->filename, swm->filename);
+				return WIMLIB_ERR_SPLIT_INVALID;
+			}
+			if (swm->hdr.part_number == 0 ||
+			    swm->hdr.part_number > total_parts)
+			{
+				ERROR("WIM `%"TS"' says it is part %u in the "
+				      "spanned set, but the part number must "
+				      "be in the range [1, %u]",
+				      swm->filename, swm->hdr.part_number, total_parts);
+				return WIMLIB_ERR_SPLIT_INVALID;
+			}
+			if (parts_to_swms[swm->hdr.part_number - 2])
+			{
+				ERROR("`%"TS"' and `%"TS"' are both marked as "
+				      "part %u of %u in the spanned set",
+				      parts_to_swms[swm->hdr.part_number - 2]->filename,
+				      swm->filename,
+				      swm->hdr.part_number,
+				      total_parts);
+				return WIMLIB_ERR_SPLIT_INVALID;
+			} else {
+				parts_to_swms[swm->hdr.part_number - 2] = swm;
+			}
+		}
+	}
+	return 0;
+}
+
 /* API function documented in wimlib.h  */
 WIMLIBAPI int
 wimlib_join(const tchar * const *swm_names,
@@ -82,10 +190,17 @@ wimlib_join(const tchar * const *swm_names,
 	if (ret)
 		goto out_free_swms;
 
-	merge_lookup_tables(swm0, additional_swms, num_additional_swms);
+	ret = wimlib_reference_resources(swm0, additional_swms,
+					 num_additional_swms, 0);
+	if (ret)
+		goto out_free_swms;
 
+	/* It is reasonably safe to provide, WIMLIB_WRITE_FLAG_STREAMS_OK, as we
+	 * have verified that the specified split WIM parts form a spanned set.
+	 */
 	ret = wimlib_write(swm0, output_path, WIMLIB_ALL_IMAGES,
-			   wim_write_flags, 1, progress_func);
+			   wim_write_flags | WIMLIB_WRITE_FLAG_STREAMS_OK,
+			   1, progress_func);
 out_free_swms:
 	for (i = 0; i < num_additional_swms; i++)
 		wimlib_free(additional_swms[i]);