X-Git-Url: https://wimlib.net/git/?p=wimlib;a=blobdiff_plain;f=src%2Fwrite.c;h=f5a9935dff47ea146780c6444812aa80eda541c8;hp=7f13630cceec991aea303afd27b74f0b1b67f888;hb=5260cf0b5649fc25b9d69a97f9604a3be257e13e;hpb=05bbdd83d65782a9872b579505e1299587cd2009

diff --git a/src/write.c b/src/write.c
index 7f13630c..f5a9935d 100644
--- a/src/write.c
+++ b/src/write.c
@@ -6,7 +6,7 @@
  */
 
 /*
- * Copyright (C) 2012, 2013, 2014, 2015 Eric Biggers
+ * Copyright (C) 2012-2016 Eric Biggers
  *
  * This file is free software; you can redistribute it and/or modify it under
  * the terms of the GNU Lesser General Public License as published by the Free
@@ -288,7 +288,7 @@ write_pwm_blob_header(const struct blob_descriptor *blob,
 	blob_hdr.flags = cpu_to_le32(reshdr_flags);
 	ret = full_write(out_fd, &blob_hdr, sizeof(blob_hdr));
 	if (ret)
-		ERROR_WITH_ERRNO("Write error");
+		ERROR_WITH_ERRNO("Error writing blob header to WIM file");
 	return ret;
 }
 
@@ -461,8 +461,11 @@ begin_chunk_table(struct write_blobs_ctx *ctx, u64 res_expected_size)
 			reserve_size += sizeof(struct alt_chunk_table_header_disk);
 		memset(ctx->chunk_csizes, 0, reserve_size);
 		ret = full_write(ctx->out_fd, ctx->chunk_csizes, reserve_size);
-		if (ret)
+		if (ret) {
+			ERROR_WITH_ERRNO("Error reserving space for chunk "
+					 "table in WIM file");
 			return ret;
+		}
 	}
 	return 0;
 }
@@ -588,7 +591,7 @@ end_chunk_table(struct write_blobs_ctx *ctx, u64 res_actual_size,
 	return 0;
 
 write_error:
-	ERROR_WITH_ERRNO("Write error");
+	ERROR_WITH_ERRNO("Error writing chunk table to WIM file");
 	return ret;
 }
 
@@ -639,6 +642,7 @@ do_done_with_blob(struct blob_descriptor *blob,
 {
 	int ret;
 	struct wim_inode *inode;
+	const tchar *path;
 	tchar *cookie1;
 	tchar *cookie2;
 
@@ -652,10 +656,12 @@ do_done_with_blob(struct blob_descriptor *blob,
 	if (--inode->i_num_remaining_streams > 0)
 		return 0;
 
-	cookie1 = progress_get_streamless_path(blob->file_on_disk);
-	cookie2 = progress_get_win32_path(blob->file_on_disk);
+	path = blob_file_path(blob);
 
-	ret = done_with_file(blob->file_on_disk, progfunc, progctx);
+	cookie1 = progress_get_streamless_path(path);
+	cookie2 = progress_get_win32_path(path);
+
+	ret = done_with_file(path, progfunc, progctx);
 
 	progress_put_win32_path(cookie2);
 	progress_put_streamless_path(cookie1);
@@ -823,7 +829,7 @@ should_rewrite_blob_uncompressed(const struct write_blobs_ctx *ctx,
 	 * Exception: if the compressed size happens to be *exactly* the same as
 	 * the uncompressed size, then the blob *must* be written uncompressed
 	 * in order to remain compatible with the Windows Overlay Filesystem
-	 * Filter Driver (WOF).
+	 * filter driver (WOF).
 	 *
 	 * TODO: we are currently assuming that the optimization for
 	 * single-chunk resources in maybe_rewrite_blob_uncompressed() prevents
@@ -980,7 +986,7 @@ write_chunk(struct write_blobs_ctx *ctx, const void *cchunk,
 				       completed_blob_count, false);
 
 write_error:
-	ERROR_WITH_ERRNO("Write error");
+	ERROR_WITH_ERRNO("Error writing chunk data to WIM file");
 	return ret;
 }
 
@@ -1237,12 +1243,18 @@ write_raw_copy_resource(struct wim_resource_descriptor *in_rdesc,
 
 			ret = full_pread(in_fd, buf, bytes_to_read,
 					 cur_read_offset);
-			if (ret)
+			if (ret) {
+				ERROR_WITH_ERRNO("Error reading raw data "
+						 "from WIM file");
 				return ret;
+			}
 
 			ret = full_write(out_fd, buf, bytes_to_read);
-			if (ret)
+			if (ret) {
+				ERROR_WITH_ERRNO("Error writing raw data "
+						 "to WIM file");
 				return ret;
+			}
 
 			cur_read_offset += bytes_to_read;
 
@@ -3220,9 +3232,8 @@ out_truncate:
 			     WIMLIB_WRITE_FLAG_UNSAFE_COMPACT))) {
 		WARNING("Truncating \"%"TS"\" to its original size "
 			"(%"PRIu64" bytes)", wim->filename, old_wim_end);
-		/* Return value of ftruncate() is ignored because this is
-		 * already an error path.  */
-		(void)ftruncate(wim->out_fd.fd, old_wim_end);
+		if (ftruncate(wim->out_fd.fd, old_wim_end))
+			WARNING_WITH_ERRNO("Failed to truncate WIM file!");
 	}
 out_restore_hdr:
 	(void)write_wim_header_flags(wim->hdr.flags, &wim->out_fd);