X-Git-Url: https://wimlib.net/git/?a=blobdiff_plain;f=src%2Fjoin.c;h=f61eaaf5eebdc6cc92f37e88dd7c8ac34d8b9df5;hb=edf0f7f5331e626b55819dcafa88e202cdb82a32;hp=1045fc7187fccf739643547b412a264b17f626da;hpb=da295f258b60e1593de305385c0669eac4b76644;p=wimlib

diff --git a/src/join.c b/src/join.c
index 1045fc71..f61eaaf5 100644
--- a/src/join.c
+++ b/src/join.c
@@ -38,17 +38,17 @@
  *		     correspond to a spanned set.
  *
  * @wim:
- * 	Part 1 of the set.
+ *	Part 1 of the set.
  *
  * @additional_swms:
- * 	All parts of the set other than part 1.
+ *	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.
+ *	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.
+ *	0 on success; WIMLIB_ERR_SPLIT_INVALID if the set is not valid.
  */
 static int
 verify_swm_set(WIMStruct *wim, WIMStruct **additional_swms,
@@ -56,6 +56,7 @@ verify_swm_set(WIMStruct *wim, WIMStruct **additional_swms,
 {
 	unsigned total_parts = wim->hdr.total_parts;
 	int ctype;
+	u32 chunk_size;
 	const u8 *guid;
 
 	if (total_parts != num_additional_swms + 1) {
@@ -83,9 +84,10 @@ verify_swm_set(WIMStruct *wim, WIMStruct **additional_swms,
 		}
 	}
 
-	/* keep track of ctype and guid just to make sure they are the same for
-	 * all the WIMs. */
+	/* Keep track of the compression type, chunk size, and GUID to make sure
+	 * they are the same for all the WIMs.  */
 	ctype = wim->compression_type;
+	chunk_size = wim->chunk_size;
 	guid = wim->hdr.guid;
 
 	{
@@ -104,7 +106,13 @@ verify_swm_set(WIMStruct *wim, WIMStruct **additional_swms,
 				      "compression type");
 				return WIMLIB_ERR_SPLIT_INVALID;
 			}
-			if (memcmp(guid, swm->hdr.guid, WIM_GID_LEN) != 0) {
+			if (swm->chunk_size != chunk_size &&
+			    ctype != WIMLIB_COMPRESSION_TYPE_NONE) {
+				ERROR("The split WIMs do not all have the same "
+				      "chunk size");
+				return WIMLIB_ERR_SPLIT_INVALID;
+			}
+			if (memcmp(guid, swm->hdr.guid, WIM_GUID_LEN) != 0) {
 				ERROR("The split WIMs do not all have the same "
 				      "GUID");
 				return WIMLIB_ERR_SPLIT_INVALID;
@@ -157,13 +165,12 @@ wimlib_join(const tchar * const *swm_names,
 	WIMStruct **additional_swms;
 	unsigned num_additional_swms;
 
-	swm_open_flags |= WIMLIB_OPEN_FLAG_SPLIT_OK;
-
 	if (num_swms < 1 || num_swms > 0xffff)
 		return WIMLIB_ERR_INVALID_PARAM;
 	num_additional_swms = num_swms - 1;
 
-	additional_swms = CALLOC(num_additional_swms, sizeof(additional_swms[0]));
+	additional_swms = CALLOC((num_additional_swms + 1),
+				 sizeof(additional_swms[0]));
 	if (!additional_swms)
 		return WIMLIB_ERR_NOMEM;
 
@@ -182,6 +189,7 @@ wimlib_join(const tchar * const *swm_names,
 	}
 
 	if (!swm0) {
+		ERROR("Part 1 of the split WIM was not specified!");
 		ret = WIMLIB_ERR_SPLIT_INVALID;
 		goto out_free_swms;
 	}
@@ -195,12 +203,16 @@ wimlib_join(const tchar * const *swm_names,
 	if (ret)
 		goto out_free_swms;
 
+	swm0->guid_set_explicitly = 1;
+
+	/* 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);
-	wimlib_unreference_resources(swm0, additional_swms,
-				     num_additional_swms);
+			   wim_write_flags | WIMLIB_WRITE_FLAG_STREAMS_OK,
+			   1, progress_func);
 out_free_swms:
-	for (i = 0; i < num_additional_swms; i++)
+	for (i = 0; i < num_additional_swms + 1; i++)
 		wimlib_free(additional_swms[i]);
 	FREE(additional_swms);
 	wimlib_free(swm0);