]> wimlib.net Git - wimlib/blobdiff - src/split.c
Remove some unused variables
[wimlib] / src / split.c
index 649a9bf2d53fd6284be7a6b6f4d32254dcb76f22..296b03fafc6bbc585295cdad3f36e657aad026e3 100644 (file)
@@ -5,7 +5,7 @@
  */
 
 /*
- * Copyright (C) 2012 Eric Biggers
+ * Copyright (C) 2012, 2013 Eric Biggers
  *
  * This file is part of wimlib, a library for working with WIM files.
  *
  * along with wimlib; if not, see http://www.gnu.org/licenses/.
  */
 
-#include "wimlib_internal.h"
-#include "lookup_table.h"
-#include "xml.h"
-#include "io.h"
+#ifdef HAVE_CONFIG_H
+#  include "config.h"
+#endif
+
+#include "wimlib.h"
+#include "wimlib/endianness.h"
+#include "wimlib/error.h"
+#include "wimlib/file_io.h"
+#include "wimlib/lookup_table.h"
+#include "wimlib/metadata.h"
+#include "wimlib/types.h"
+#include "wimlib/write.h"
+#include "wimlib/list.h"
+
+#include <fcntl.h> /* for open() */
+#include <unistd.h> /* for close() */
 
 struct split_args {
        WIMStruct *w;
-       char *swm_base_name;
+       tchar *swm_base_name;
        size_t swm_base_name_len;
-       const char *swm_suffix;
+       const tchar *swm_suffix;
        struct list_head lte_list;
-       int    cur_part_number;
-       int    write_flags;
-       long   size_remaining;
+       int cur_part_number;
+       int write_flags;
+       long size_remaining;
        size_t part_size;
        wimlib_progress_func_t progress_func;
        union wimlib_progress_info progress;
 };
 
-static int finish_swm(WIMStruct *w, struct list_head *lte_list,
-                     int write_flags, wimlib_progress_func_t progress_func)
+static int
+finish_swm(WIMStruct *w, struct list_head *lte_list,
+          int write_flags, wimlib_progress_func_t progress_func)
 {
-       off_t lookup_table_offset = ftello(w->out_fp);
        int ret;
-       struct lookup_table_entry *lte;
-
-       list_for_each_entry(lte, lte_list, staging_list) {
-               ret = write_lookup_table_entry(lte, w->out_fp);
-               if (ret != 0)
-                       return ret;
-       }
-
-       off_t xml_data_offset = ftello(w->out_fp);
 
-       if (lookup_table_offset == -1 || xml_data_offset == -1)
-               return WIMLIB_ERR_WRITE;
-       w->hdr.lookup_table_res_entry.offset = lookup_table_offset;
-       w->hdr.lookup_table_res_entry.size =
-                               xml_data_offset - lookup_table_offset;
-       w->hdr.lookup_table_res_entry.original_size =
-                               xml_data_offset - lookup_table_offset;
-       w->hdr.lookup_table_res_entry.flags = WIM_RESHDR_FLAG_METADATA;
+       ret = write_lookup_table_from_stream_list(lte_list, w->out_fd,
+                                                 &w->hdr.lookup_table_res_entry);
+       if (ret)
+               return ret;
        return finish_write(w, WIMLIB_ALL_IMAGES,
                            write_flags | WIMLIB_WRITE_FLAG_NO_LOOKUP_TABLE,
                            progress_func);
 }
 
-static int copy_resource_to_swm(struct lookup_table_entry *lte, void *__args)
+static int
+copy_resource_to_swm(struct wim_lookup_table_entry *lte, void *_args)
 {
-       struct split_args *args = (struct split_args*)__args;
+       struct split_args *args = (struct split_args*)_args;
        WIMStruct *w = args->w;
        int ret;
 
@@ -88,7 +88,7 @@ static int copy_resource_to_swm(struct lookup_table_entry *lte, void *__args)
 
                ret = finish_swm(w, &args->lte_list, args->write_flags,
                                 args->progress_func);
-               if (ret != 0)
+               if (ret)
                        return ret;
 
                if (args->progress_func) {
@@ -99,8 +99,8 @@ static int copy_resource_to_swm(struct lookup_table_entry *lte, void *__args)
                INIT_LIST_HEAD(&args->lte_list);
                args->cur_part_number++;
 
-               sprintf(args->swm_base_name + args->swm_base_name_len, "%d%s",
-                       args->cur_part_number, args->swm_suffix);
+               tsprintf(args->swm_base_name + args->swm_base_name_len, T("%d%"TS),
+                        args->cur_part_number, args->swm_suffix);
 
                w->hdr.part_number = args->cur_part_number;
 
@@ -111,30 +111,31 @@ static int copy_resource_to_swm(struct lookup_table_entry *lte, void *__args)
                }
 
                ret = begin_write(w, args->swm_base_name, args->write_flags);
-               if (ret != 0)
+               if (ret)
                        return ret;
                args->size_remaining = args->part_size;
        }
        args->size_remaining -= lte->resource_entry.size;
        args->progress.split.completed_bytes += lte->resource_entry.size;
-       list_add(&lte->staging_list, &args->lte_list);
+       list_add_tail(&lte->swm_stream_list, &args->lte_list);
        return copy_resource(lte, w);
 }
 
-/* Splits the WIM file @wimfile into multiple parts prefixed by @swm_name with
- * size at most @part_size. */
-WIMLIBAPI int wimlib_split(WIMStruct *w, const char *swm_name,
-                          size_t part_size, int write_flags,
-                          wimlib_progress_func_t progress_func)
+/* Splits the WIM file @w into multiple parts prefixed by @swm_name with size at
+ * most @part_size bytes. */
+WIMLIBAPI int
+wimlib_split(WIMStruct *w, const tchar *swm_name,
+            size_t part_size, int write_flags,
+            wimlib_progress_func_t progress_func)
 {
        int ret;
        struct wim_header hdr_save;
        struct split_args args;
-       const char *swm_suffix;
+       const tchar *swm_suffix;
        size_t swm_name_len;
        size_t swm_base_name_len;
 
-       if (!w || !swm_name || part_size == 0)
+       if (!swm_name || part_size == 0)
                return WIMLIB_ERR_INVALID_PARAM;
 
        if (w->hdr.total_parts != 1)
@@ -142,26 +143,26 @@ WIMLIBAPI int wimlib_split(WIMStruct *w, const char *swm_name,
 
        write_flags &= WIMLIB_WRITE_MASK_PUBLIC;
 
-       swm_name_len = strlen(swm_name);
-       char swm_base_name[swm_name_len + 20];
+       swm_name_len = tstrlen(swm_name);
+       tchar swm_base_name[swm_name_len + 20];
 
        memcpy(&hdr_save, &w->hdr, sizeof(struct wim_header));
        w->hdr.flags |= WIM_HDR_FLAG_SPANNED;
        w->hdr.boot_idx = 0;
        randomize_byte_array(w->hdr.guid, WIM_GID_LEN);
        ret = begin_write(w, swm_name, write_flags);
-       if (ret != 0)
+       if (ret)
                goto out;
 
-       memcpy(swm_base_name, swm_name, swm_name_len + 1);
+       tmemcpy(swm_base_name, swm_name, swm_name_len + 1);
 
-       swm_suffix = strchr(swm_name, '.');
+       swm_suffix = tstrchr(swm_name, T('.'));
        if (swm_suffix) {
                swm_base_name_len = swm_suffix - swm_name;
        } else {
                swm_base_name_len = swm_name_len;
-               swm_base_name[sizeof(swm_base_name) - 1] = '\0';
-               swm_suffix = &swm_base_name[sizeof(swm_base_name) - 1];
+               swm_base_name[ARRAY_LEN(swm_base_name) - 1] = T('\0');
+               swm_suffix = &swm_base_name[ARRAY_LEN(swm_base_name) - 1];
        }
 
        args.w                              = w;
@@ -184,26 +185,27 @@ WIMLIBAPI int wimlib_split(WIMStruct *w, const char *swm_name,
                              &args.progress);
        }
 
-       w->write_metadata = true;
        for (int i = 0; i < w->hdr.image_count; i++) {
-               struct lookup_table_entry *metadata_lte;
-               metadata_lte = w->image_metadata[i].metadata_lte;
+               struct wim_lookup_table_entry *metadata_lte;
+               metadata_lte = w->image_metadata[i]->metadata_lte;
                ret = copy_resource(metadata_lte, w);
-               if (ret != 0)
+               if (ret)
                        goto out;
                args.size_remaining -= metadata_lte->resource_entry.size;
                args.progress.split.completed_bytes += metadata_lte->resource_entry.size;
-               list_add(&metadata_lte->staging_list, &args.lte_list);
+               /* Careful: The metadata lookup table entries must be added in
+                * order of the images. */
+               list_add_tail(&metadata_lte->swm_stream_list, &args.lte_list);
        }
-       w->write_metadata = false;
 
-       ret = for_lookup_table_entry(w->lookup_table,
-                                    copy_resource_to_swm, &args);
-       if (ret != 0)
+       ret = for_lookup_table_entry_pos_sorted(w->lookup_table,
+                                               copy_resource_to_swm,
+                                               &args);
+       if (ret)
                goto out;
 
        ret = finish_swm(w, &args.lte_list, write_flags, progress_func);
-       if (ret != 0)
+       if (ret)
                goto out;
 
        if (progress_func) {
@@ -212,38 +214,40 @@ WIMLIBAPI int wimlib_split(WIMStruct *w, const char *swm_name,
        }
 
        /* The swms are all ready now, except the total_parts and part_number
-        * fields in their headers are wrong (we don't know the total parts
-        * until they are all written).  Fix them. */
+        * fields in their headers are wrong (since we don't know the total
+        * parts until they are all written).  Fix them. */
        int total_parts = args.cur_part_number;
        for (int i = 1; i <= total_parts; i++) {
-               const char *part_name;
+               const tchar *part_name;
+               int part_fd;
+               int ret2;
+
                if (i == 1) {
                        part_name = swm_name;
                } else {
-                       sprintf(swm_base_name + swm_base_name_len, "%d%s",
-                               i, swm_suffix);
+                       tsprintf(swm_base_name + swm_base_name_len, T("%d%"TS),
+                                i, swm_suffix);
                        part_name = swm_base_name;
                }
 
-               FILE *fp = fopen(part_name, "r+b");
-               if (!fp) {
-                       ERROR_WITH_ERRNO("Failed to open `%s'", part_name);
+               part_fd = topen(part_name, O_WRONLY | O_BINARY);
+               if (part_fd == -1) {
+                       ERROR_WITH_ERRNO("Failed to open `%"TS"'", part_name);
                        ret = WIMLIB_ERR_OPEN;
                        goto out;
                }
-               u8 buf[4];
-               put_u16(&buf[0], i);
-               put_u16(&buf[2], total_parts);
-
-               if (fseek(fp, 40, SEEK_SET) != 0 ||
-                               fwrite(buf, 1, sizeof(buf), fp) != sizeof(buf)
-                               || fclose(fp) != 0) {
-                       ERROR_WITH_ERRNO("Error overwriting header of `%s'",
-                                        part_name);
+
+               ret = write_header_part_data(i, total_parts, part_fd);
+               ret2 = close(part_fd);
+               if (ret == 0 && ret2 != 0)
                        ret = WIMLIB_ERR_WRITE;
-                       break;
+               if (ret) {
+                       ERROR_WITH_ERRNO("Error updating header of `%"TS"'",
+                                        part_name);
+                       goto out;
                }
        }
+       ret = 0;
 out:
        close_wim_writable(w);
        memcpy(&w->hdr, &hdr_save, sizeof(struct wim_header));