* along with wimlib; if not, see http://www.gnu.org/licenses/.
*/
-#include "wimlib_internal.h"
#include "dentry.h"
-#include "xml.h"
-#include "timestamp.h"
#include "lookup_table.h"
+#include "timestamp.h"
+#include "wimlib_internal.h"
+#include "xml.h"
+
#include <string.h>
-#include <time.h>
#include <libxml/parser.h>
#include <libxml/tree.h>
#include <libxml/xmlwriter.h>
+#include <libxml/encoding.h>
+#include <limits.h>
/* Structures used to form an in-memory representation of the XML data (other
* than the raw parse tree from libxml). */
char *product_suite;
char **languages;
char *default_language;
- u64 num_languages;
+ size_t num_languages;
char *system_root;
bool windows_version_exists;
struct windows_version windows_version;
};
struct image_info {
- u64 index;
- u64 dir_count;
- u64 file_count;
- u64 total_bytes;
- u64 hard_link_bytes;
- u64 creation_time;
- u64 last_modification_time;
- bool windows_info_exists;
+ int index;
+ bool windows_info_exists;
+ u64 dir_count;
+ u64 file_count;
+ u64 total_bytes;
+ u64 hard_link_bytes;
+ u64 creation_time;
+ u64 last_modification_time;
struct windows_info windows_info;
char *name;
char *description;
- char *display_name;
- char *display_description;
+ char *display_name;
+ char *display_description;
union {
- char *flags;
+ char *flags;
struct lookup_table *lookup_table;
};
};
return node->type == XML_TEXT_NODE;
}
-static inline bool node_is_attribute(xmlNode *node)
-{
- return node->type == XML_ATTRIBUTE_NODE;
-}
-
static inline bool node_name_is(xmlNode *node, const char *name)
{
/* For now, both upper case and lower case element names are accepted. */
for_node_child(string_node, child) {
if (node_is_text(child) && child->content) {
p = STRDUP(child->content);
- if (!p) {
- ERROR("Out of memory");
+ if (!p)
return WIMLIB_ERR_NOMEM;
- }
break;
}
}
/* Used to sort an array of struct image_infos by their image indices. */
static int sort_by_index(const void *p1, const void *p2)
{
- u64 index_1 = ((struct image_info*)p1)->index;
- u64 index_2 = ((struct image_info*)p1)->index;
+ int index_1 = ((const struct image_info*)p1)->index;
+ int index_2 = ((const struct image_info*)p2)->index;
if (index_1 < index_2)
return -1;
else if (index_1 > index_2)
/* Frees memory allocated inside a struct windows_info structure. */
static void destroy_windows_info(struct windows_info *windows_info)
{
- uint i;
-
FREE(windows_info->product_name);
FREE(windows_info->edition_id);
FREE(windows_info->installation_type);
+ FREE(windows_info->hal);
FREE(windows_info->product_type);
- for (i = 0; i < windows_info->num_languages; i++)
+ FREE(windows_info->product_suite);
+ for (size_t i = 0; i < windows_info->num_languages; i++)
FREE(windows_info->languages[i]);
FREE(windows_info->languages);
+ FREE(windows_info->default_language);
FREE(windows_info->system_root);
}
void free_wim_info(struct wim_info *info)
{
- uint i;
if (info) {
if (info->images) {
- for (i = 0; i < info->num_images; i++)
+ for (int i = 0; i < info->num_images; i++)
destroy_image_info(&info->images[i]);
FREE(info->images);
}
/* Reads the information from a <VERSION> element inside the <WINDOWS> element.
* */
-static void xml_read_windows_version(const xmlNode *version_node,
+static void xml_read_windows_version(const xmlNode *version_node,
struct windows_version* windows_version)
{
xmlNode *child;
if (!node_is_element(child))
continue;
if (node_name_is(child, "MAJOR"))
- windows_version->major = node_get_u64(child);
+ windows_version->major = node_get_u64(child);
else if (node_name_is(child, "MINOR"))
- windows_version->minor = node_get_u64(child);
+ windows_version->minor = node_get_u64(child);
else if (node_name_is(child, "BUILD"))
- windows_version->build = node_get_u64(child);
+ windows_version->build = node_get_u64(child);
else if (node_name_is(child, "SPBUILD"))
windows_version->sp_build = node_get_u64(child);
else if (node_name_is(child, "SPLEVEL"))
/* Reads the information from a <LANGUAGE> element inside a <WINDOWS> element.
* */
-static int xml_read_languages(const xmlNode *languages_node,
- char ***languages_ret,
- u64 *num_languages_ret,
+static int xml_read_languages(const xmlNode *languages_node,
+ char ***languages_ret,
+ size_t *num_languages_ret,
char **default_language_ret)
{
xmlNode *child;
- uint i;
- uint num_languages;
+ size_t num_languages = 0;
char **languages;
int ret;
- num_languages = 0;
for_node_child(languages_node, child)
if (node_is_element(child) && node_name_is(child, "LANGUAGE"))
num_languages++;
- languages = CALLOC(num_languages, sizeof(char*));
- if (!languages) {
- ERROR("Out of memory");
+ languages = CALLOC(num_languages, sizeof(languages[0]));
+ if (!languages)
return WIMLIB_ERR_NOMEM;
- }
*languages_ret = languages;
*num_languages_ret = num_languages;
- i = 0;
ret = 0;
for_node_child(languages_node, child) {
if (!node_is_element(child))
continue;
if (node_name_is(child, "LANGUAGE"))
- ret = node_get_string(child, &languages[i++]);
+ ret = node_get_string(child, languages++);
else if (node_name_is(child, "DEFAULT"))
ret = node_get_string(child, default_language_ret);
if (ret != 0)
- return ret;
+ break;
}
return ret;
}
/* Reads the information from a <WINDOWS> element inside an <IMAGE> element. */
-static int xml_read_windows_info(const xmlNode *windows_node,
+static int xml_read_windows_info(const xmlNode *windows_node,
struct windows_info *windows_info)
{
xmlNode *child;
if (node_name_is(child, "ARCH")) {
windows_info->arch = node_get_u64(child);
} else if (node_name_is(child, "PRODUCTNAME")) {
- ret = node_get_string(child,
+ ret = node_get_string(child,
&windows_info->product_name);
} else if (node_name_is(child, "EDITIONID")) {
- ret = node_get_string(child,
+ ret = node_get_string(child,
&windows_info->edition_id);
} else if (node_name_is(child, "INSTALLATIONTYPE")) {
- ret = node_get_string(child,
+ ret = node_get_string(child,
&windows_info->installation_type);
} else if (node_name_is(child, "PRODUCTTYPE")) {
- ret = node_get_string(child,
+ ret = node_get_string(child,
&windows_info->product_type);
} else if (node_name_is(child, "PRODUCTSUITE")) {
- ret = node_get_string(child,
+ ret = node_get_string(child,
&windows_info->product_suite);
} else if (node_name_is(child, "LANGUAGES")) {
- ret = xml_read_languages(child,
+ ret = xml_read_languages(child,
&windows_info->languages,
&windows_info->num_languages,
&windows_info->default_language);
} else if (node_name_is(child, "VERSION")) {
- xml_read_windows_version(child,
+ xml_read_windows_version(child,
&windows_info->windows_version);
windows_info->windows_version_exists = true;
} else if (node_name_is(child, "SYSTEMROOT")) {
} else if (node_name_is(child, "HAL")) {
ret = node_get_string(child, &windows_info->hal);
}
-
if (ret != 0)
return ret;
}
}
/* Reads the information from an <IMAGE> element. */
-static int xml_read_image_info(xmlNode *image_node,
+static int xml_read_image_info(xmlNode *image_node,
struct image_info *image_info)
{
xmlNode *child;
xmlChar *index_prop;
int ret;
-
+
index_prop = xmlGetProp(image_node, "INDEX");
if (index_prop) {
- char *tmp;
- image_info->index = strtoul(index_prop, &tmp, 10);
+ image_info->index = atoi(index_prop);
FREE(index_prop);
} else {
- image_info->index = 0;
+ image_info->index = 1;
}
ret = 0;
return ret;
}
if (!image_info->name) {
- WARNING("Image with index %"PRIu64" has no name",
- image_info->index);
- image_info->name = MALLOC(1);
- if (!image_info->name) {
- ERROR("Out of memory");
+ char *empty_name;
+ WARNING("Image with index %d has no name", image_info->index);
+ empty_name = MALLOC(1);
+ if (!empty_name)
return WIMLIB_ERR_NOMEM;
- }
- image_info->name[0] = '\0';
+ *empty_name = '\0';
+ image_info->name = empty_name;
}
return ret;
}
struct wim_info *wim_info;
xmlNode *child;
int ret;
- uint num_images;
- struct image_info *cur_image_info;
+ int num_images;
+ int i;
wim_info = CALLOC(1, sizeof(struct wim_info));
- if (!wim_info) {
- ERROR("Out of memory");
+ if (!wim_info)
return WIMLIB_ERR_NOMEM;
- }
/* Count how many images there are. */
num_images = 0;
- for_node_child(wim_node, child)
- if (node_is_element(child) && node_name_is(child, "IMAGE"))
+ for_node_child(wim_node, child) {
+ if (node_is_element(child) && node_name_is(child, "IMAGE")) {
+ if (num_images == INT_MAX) {
+ return WIMLIB_ERR_IMAGE_COUNT;
+ }
num_images++;
+ }
+ }
- if (num_images == 0)
- goto done;
+ if (num_images > 0) {
+ /* Allocate the array of struct image_infos and fill them in. */
+ wim_info->images = CALLOC(num_images, sizeof(wim_info->images[0]));
+ if (!wim_info->images) {
+ ret = WIMLIB_ERR_NOMEM;
+ goto err;
+ }
+ wim_info->num_images = num_images;
+ i = 0;
+ for_node_child(wim_node, child) {
+ if (!node_is_element(child))
+ continue;
+ if (node_name_is(child, "IMAGE")) {
+ DEBUG("Found <IMAGE> tag");
+ ret = xml_read_image_info(child,
+ &wim_info->images[i]);
+ if (ret != 0)
+ goto err;
+ i++;
+ } else if (node_name_is(child, "TOTALBYTES")) {
+ wim_info->total_bytes = node_get_u64(child);
+ }
+ }
- /* Allocate the array of struct image_infos and fill them in. */
- wim_info->images = CALLOC(num_images, sizeof(wim_info->images[0]));
- if (!wim_info->images) {
- ret = WIMLIB_ERR_NOMEM;
- ERROR("Out of memory!");
- goto err;
- }
- wim_info->num_images = num_images;
- cur_image_info = wim_info->images;
- for_node_child(wim_node, child) {
- if (!node_is_element(child))
- continue;
- if (node_name_is(child, "IMAGE")) {
- DEBUG("Found <IMAGE> tag");
- ret = xml_read_image_info(child, cur_image_info++);
- if (ret != 0)
- goto err;
- } else if (node_name_is(child, "TOTALBYTES")) {
- wim_info->total_bytes = node_get_u64(child);
+ /* Sort the array of image info by image index. */
+ qsort(wim_info->images, num_images,
+ sizeof(struct image_info), sort_by_index);
+
+ /* Make sure the image indices make sense */
+ for (i = 0; i < num_images; i++) {
+ if (wim_info->images[i].index != i + 1) {
+ ERROR("WIM images are not indexed [1...%d] "
+ "in XML data as expected",
+ num_images);
+ return WIMLIB_ERR_IMAGE_COUNT;
+ }
}
- }
- /* Sort the array of struct image_infos by image index. */
- qsort(wim_info->images, wim_info->num_images,
- sizeof(struct image_info), sort_by_index);
-done:
+ }
*wim_info_ret = wim_info;
return 0;
err:
return ret;
}
-/* Prints the information contained in a struct windows_info structure. */
+/* Prints the information contained in a `struct windows_info'. */
static void print_windows_info(const struct windows_info *windows_info)
{
- uint i;
const struct windows_version *windows_version;
printf("Architecture: %s\n", get_arch(windows_info->arch));
- printf("Product Name: %s\n", windows_info->product_name);
- printf("Edition ID: %s\n", windows_info->edition_id);
- printf("Installation Type: %s\n", windows_info->installation_type);
+
+ if (windows_info->product_name)
+ printf("Product Name: %s\n",
+ windows_info->product_name);
+
+ if (windows_info->edition_id)
+ printf("Edition ID: %s\n",
+ windows_info->edition_id);
+
+ if (windows_info->installation_type)
+ printf("Installation Type: %s\n",
+ windows_info->installation_type);
+
if (windows_info->hal)
- printf("HAL: %s\n", windows_info->hal);
- printf("Product Type: %s\n", windows_info->product_type);
+ printf("HAL: %s\n",
+ windows_info->hal);
+
+ if (windows_info->product_type)
+ printf("Product Type: %s\n",
+ windows_info->product_type);
+
if (windows_info->product_suite)
- printf("Product Suite: %s\n", windows_info->product_suite);
+ printf("Product Suite: %s\n",
+ windows_info->product_suite);
+
printf("Languages: ");
- for (i = 0; i < windows_info->num_languages; i++) {
+ for (size_t i = 0; i < windows_info->num_languages; i++) {
fputs(windows_info->languages[i], stdout);
putchar(' ');
}
putchar('\n');
- printf("Default Language: %s\n", windows_info->default_language);
- printf("System Root: %s\n", windows_info->system_root);
+ if (windows_info->default_language)
+ printf("Default Language: %s\n",
+ windows_info->default_language);
+ if (windows_info->system_root)
+ printf("System Root: %s\n",
+ windows_info->system_root);
if (windows_info->windows_version_exists) {
windows_version = &windows_info->windows_version;
- printf("Major Version: %"PRIu64"\n",
- windows_version->major);
- printf("Minor Version: %"PRIu64"\n",
- windows_version->minor);
- printf("Build: %"PRIu64"\n",
- windows_version->build);
- printf("Service Pack Build: %"PRIu64"\n",
- windows_version->sp_build);
- printf("Service Pack Level: %"PRIu64"\n",
- windows_version->sp_level);
+ printf("Major Version: %"PRIu64"\n",
+ windows_version->major);
+ printf("Minor Version: %"PRIu64"\n",
+ windows_version->minor);
+ printf("Build: %"PRIu64"\n",
+ windows_version->build);
+ printf("Service Pack Build: %"PRIu64"\n",
+ windows_version->sp_build);
+ printf("Service Pack Level: %"PRIu64"\n",
+ windows_version->sp_level);
}
}
-/* Writes the information contained in a struct windows_version structure to the XML
- * document being constructed in memory. This is the <VERSION> element inside
- * the <WINDOWS> element. */
-static int xml_write_windows_version(xmlTextWriter *writer,
- const struct windows_version *version)
+/* Writes the information contained in a `struct windows_version' to the XML
+ * document being written. This is the <VERSION> element inside the <WINDOWS>
+ * element. */
+static int xml_write_windows_version(xmlTextWriter *writer,
+ const struct windows_version *version)
{
int rc;
rc = xmlTextWriterStartElement(writer, "VERSION");
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "MAJOR", "%"PRIu64,
- version->major);
+ rc = xmlTextWriterWriteFormatElement(writer, "MAJOR", "%"PRIu64,
+ version->major);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "MINOR", "%"PRIu64,
- version->minor);
+ rc = xmlTextWriterWriteFormatElement(writer, "MINOR", "%"PRIu64,
+ version->minor);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "BUILD", "%"PRIu64,
- version->build);
+ rc = xmlTextWriterWriteFormatElement(writer, "BUILD", "%"PRIu64,
+ version->build);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "SPBUILD", "%"PRIu64,
- version->sp_build);
+ rc = xmlTextWriterWriteFormatElement(writer, "SPBUILD", "%"PRIu64,
+ version->sp_build);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "SPLEVEL", "%"PRIu64,
- version->sp_level);
+ rc = xmlTextWriterWriteFormatElement(writer, "SPLEVEL", "%"PRIu64,
+ version->sp_level);
if (rc < 0)
return rc;
return xmlTextWriterEndElement(writer); /* </VERSION> */
}
-/* Writes the information contained in a struct windows_info structure to the XML
- * document being constructed in memory. This is the <WINDOWS> element. */
-static int xml_write_windows_info(xmlTextWriter *writer,
- const struct windows_info *windows_info)
+/* Writes the information contained in a `struct windows_info' to the XML
+ * document being written. This is the <WINDOWS> element. */
+static int xml_write_windows_info(xmlTextWriter *writer,
+ const struct windows_info *windows_info)
{
int rc;
rc = xmlTextWriterStartElement(writer, "WINDOWS");
if (rc < 0)
return rc;
-
- rc = xmlTextWriterWriteFormatElement(writer, "ARCH", "%"PRIu64,
- windows_info->arch);
+ rc = xmlTextWriterWriteFormatElement(writer, "ARCH", "%"PRIu64,
+ windows_info->arch);
if (rc < 0)
return rc;
-
+
if (windows_info->product_name) {
- rc = xmlTextWriterWriteElement(writer, "PRODUCTNAME",
- windows_info->product_name);
+ rc = xmlTextWriterWriteElement(writer, "PRODUCTNAME",
+ windows_info->product_name);
if (rc < 0)
return rc;
}
if (windows_info->edition_id) {
- rc = xmlTextWriterWriteElement(writer, "EDITIONID",
- windows_info->edition_id);
+ rc = xmlTextWriterWriteElement(writer, "EDITIONID",
+ windows_info->edition_id);
if (rc < 0)
return rc;
}
if (windows_info->installation_type) {
- rc = xmlTextWriterWriteElement(writer, "INSTALLATIONTYPE",
- windows_info->installation_type);
+ rc = xmlTextWriterWriteElement(writer, "INSTALLATIONTYPE",
+ windows_info->installation_type);
if (rc < 0)
return rc;
}
if (windows_info->hal) {
- rc = xmlTextWriterWriteElement(writer, "HAL",
- windows_info->hal);
+ rc = xmlTextWriterWriteElement(writer, "HAL",
+ windows_info->hal);
if (rc < 0)
return rc;
}
- if (windows_info->system_root) {
- rc = xmlTextWriterWriteElement(writer, "SYSTEMROOT",
- windows_info->system_root);
- if (rc < 0)
- return rc;
- }
-
if (windows_info->product_type) {
- rc = xmlTextWriterWriteElement(writer, "PRODUCTTYPE",
- windows_info->product_type);
+ rc = xmlTextWriterWriteElement(writer, "PRODUCTTYPE",
+ windows_info->product_type);
if (rc < 0)
return rc;
}
if (windows_info->product_suite) {
- rc = xmlTextWriterWriteElement(writer, "PRODUCTSUITE",
- windows_info->product_suite);
- if (rc < 0)
- return rc;
+ rc = xmlTextWriterWriteElement(writer, "PRODUCTSUITE",
+ windows_info->product_suite);
+ if (rc < 0)
+ return rc;
}
if (windows_info->num_languages) {
if (rc < 0)
return rc;
- for (int i = 0; i < windows_info->num_languages; i++) {
- rc = xmlTextWriterWriteElement(writer, "LANGUAGE",
- windows_info->languages[i]);
+ for (size_t i = 0; i < windows_info->num_languages; i++) {
+ rc = xmlTextWriterWriteElement(writer, "LANGUAGE",
+ windows_info->languages[i]);
if (rc < 0)
return rc;
}
- rc = xmlTextWriterWriteElement(writer, "DEFAULT",
- windows_info->default_language);
+ rc = xmlTextWriterWriteElement(writer, "DEFAULT",
+ windows_info->default_language);
if (rc < 0)
return rc;
return rc;
}
+ if (windows_info->system_root) {
+ rc = xmlTextWriterWriteElement(writer, "SYSTEMROOT",
+ windows_info->system_root);
+ if (rc < 0)
+ return rc;
+ }
+
return xmlTextWriterEndElement(writer); /* </WINDOWS> */
}
/* Writes a time element to the XML document being constructed in memory. */
-static int xml_write_time(xmlTextWriter *writer, const char *element_name,
- u64 time)
+static int xml_write_time(xmlTextWriter *writer, const char *element_name,
+ u64 time)
{
int rc;
rc = xmlTextWriterStartElement(writer, element_name);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "HIGHPART",
- "0x%"PRIX32, (u32)(time >> 32));
+ rc = xmlTextWriterWriteFormatElement(writer, "HIGHPART",
+ "0x%08"PRIX32, (u32)(time >> 32));
if (rc < 0)
return rc;
rc = xmlTextWriterWriteFormatElement(writer, "LOWPART",
- "0x%"PRIX32, (u32)time);
+ "0x%08"PRIX32, (u32)time);
if (rc < 0)
return rc;
return 0;
}
-
/* Writes an <IMAGE> element to the XML document. */
-static int xml_write_image_info(xmlTextWriter *writer,
+static int xml_write_image_info(xmlTextWriter *writer,
const struct image_info *image_info)
{
int rc;
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatAttribute(writer, "INDEX", "%"PRIu64,
- image_info->index);
+ rc = xmlTextWriterWriteFormatAttribute(writer, "INDEX", "%d",
+ image_info->index);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "DIRCOUNT", "%"PRIu64,
- image_info->dir_count);
+ rc = xmlTextWriterWriteFormatElement(writer, "DIRCOUNT", "%"PRIu64,
+ image_info->dir_count);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "FILECOUNT", "%"PRIu64,
- image_info->file_count);
+ rc = xmlTextWriterWriteFormatElement(writer, "FILECOUNT", "%"PRIu64,
+ image_info->file_count);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "TOTALBYTES", "%"PRIu64,
- image_info->total_bytes);
+ rc = xmlTextWriterWriteFormatElement(writer, "TOTALBYTES", "%"PRIu64,
+ image_info->total_bytes);
if (rc < 0)
return rc;
- rc = xmlTextWriterWriteFormatElement(writer, "HARDLINKBYTES", "%"PRIu64,
- image_info->hard_link_bytes);
+ rc = xmlTextWriterWriteFormatElement(writer, "HARDLINKBYTES", "%"PRIu64,
+ image_info->hard_link_bytes);
if (rc < 0)
return rc;
- rc = xml_write_time(writer, "CREATIONTIME",
- image_info->creation_time);
+ rc = xml_write_time(writer, "CREATIONTIME", image_info->creation_time);
if (rc < 0)
return rc;
- rc = xml_write_time(writer, "LASTMODIFICATIONTIME",
- image_info->last_modification_time);
+ rc = xml_write_time(writer, "LASTMODIFICATIONTIME",
+ image_info->last_modification_time);
if (rc < 0)
return rc;
rc = xml_write_windows_info(writer, &image_info->windows_info);
if (rc < 0)
return rc;
- } else {
- DEBUG("<WINDOWS> tag does not exist.");
}
if (image_info->name) {
- rc = xmlTextWriterWriteElement(writer, "NAME", image_info->name);
+ rc = xmlTextWriterWriteElement(writer, "NAME",
+ image_info->name);
if (rc < 0)
return rc;
}
+
if (image_info->description) {
- rc = xmlTextWriterWriteElement(writer, "DESCRIPTION",
- image_info->description);
+ rc = xmlTextWriterWriteElement(writer, "DESCRIPTION",
+ image_info->description);
if (rc < 0)
return rc;
}
if (image_info->display_name) {
- rc = xmlTextWriterWriteElement(writer, "DISPLAYNAME",
- image_info->display_name);
+ rc = xmlTextWriterWriteElement(writer, "DISPLAYNAME",
+ image_info->display_name);
if (rc < 0)
return rc;
}
if (image_info->display_description) {
- rc = xmlTextWriterWriteElement(writer, "DISPLAYDESCRIPTION",
- image_info->display_description);
+ rc = xmlTextWriterWriteElement(writer, "DISPLAYDESCRIPTION",
+ image_info->display_description);
if (rc < 0)
return rc;
}
if (image_info->flags) {
rc = xmlTextWriterWriteElement(writer, "FLAGS",
- image_info->flags);
+ image_info->flags);
if (rc < 0)
return rc;
}
return &images[wim_info->num_images - 1];
}
-static int clone_windows_info(const struct windows_info *old,
+static int clone_windows_info(const struct windows_info *old,
struct windows_info *new)
{
- uint i;
-
if (old->product_name && !(new->product_name = STRDUP(old->product_name)))
return WIMLIB_ERR_NOMEM;
if (old->edition_id && !(new->edition_id = STRDUP(old->edition_id)))
return WIMLIB_ERR_NOMEM;
- if (old->installation_type && !(new->installation_type =
+ if (old->installation_type && !(new->installation_type =
STRDUP(old->installation_type)))
return WIMLIB_ERR_NOMEM;
if (old->hal && !(new->hal = STRDUP(old->hal)))
if (!new->languages)
return WIMLIB_ERR_NOMEM;
new->num_languages = old->num_languages;
- for (i = 0; i < new->num_languages; i++) {
+ for (size_t i = 0; i < new->num_languages; i++) {
if (!old->languages[i])
continue;
new->languages[i] = STRDUP(old->languages[i]);
return WIMLIB_ERR_NOMEM;
}
}
- if (old->default_language &&
+ if (old->default_language &&
!(new->default_language = STRDUP(old->default_language)))
return WIMLIB_ERR_NOMEM;
if (old->system_root && !(new->system_root = STRDUP(old->system_root)))
if (old->windows_info_exists) {
new->windows_info_exists = true;
- return clone_windows_info(&old->windows_info,
+ return clone_windows_info(&old->windows_info,
&new->windows_info);
}
return 0;
}
-/* Copies the XML information for an image between WIM files.
+/* Copies the XML information for an image between WIM files.
*
* @dest_image_name and @dest_image_description are ignored if they are NULL;
* otherwise, they are used to override the image name and/or image description
- * from the XML data in the source WIM file. */
-int xml_export_image(const struct wim_info *old_wim_info,
- int image,
- struct wim_info **new_wim_info_p,
- const char *dest_image_name,
+ * from the XML data in the source WIM file.
+ *
+ * On failure, WIMLIB_ERR_NOMEM is returned and no changes are made. Otherwise,
+ * 0 is returned and the WIM information at *new_wim_info_p is modified.
+ */
+int xml_export_image(const struct wim_info *old_wim_info,
+ int image,
+ struct wim_info **new_wim_info_p,
+ const char *dest_image_name,
const char *dest_image_description)
{
struct wim_info *new_wim_info;
DEBUG("Copying XML data between WIM files for source image %d.", image);
+ wimlib_assert(old_wim_info != NULL);
wimlib_assert(image >= 1 && image <= old_wim_info->num_images);
if (*new_wim_info_p) {
ret = clone_image_info(&old_wim_info->images[image - 1], image_info);
if (ret != 0)
- goto err;
+ goto err_destroy_image_info;
image_info->index = new_wim_info->num_images;
FREE(image_info->name);
image_info->name = STRDUP(dest_image_name);
if (!image_info->name)
- goto err;
+ goto err_destroy_image_info;
}
if (dest_image_description) {
FREE(image_info->description);
image_info->description = STRDUP(dest_image_description);
if (!image_info->description)
- goto err;
+ goto err_destroy_image_info;
}
*new_wim_info_p = new_wim_info;
return 0;
+err_destroy_image_info:
+ destroy_image_info(image_info);
err:
- ERROR("Out of memory");
- free_wim_info(new_wim_info);
+ if (new_wim_info != *new_wim_info_p)
+ free_wim_info(new_wim_info);
return WIMLIB_ERR_NOMEM;
}
void xml_delete_image(struct wim_info **wim_info_p, int image)
{
struct wim_info *wim_info;
- int i;
DEBUG("Deleting image %d from the XML data.", image);
-
- wim_info = *wim_info_p;
- wimlib_assert(wim_info);
- wimlib_assert(image >= 1 && image <= wim_info->num_images);
+ wim_info = *wim_info_p;
destroy_image_info(&wim_info->images[image - 1]);
- for (i = image - 1; i < wim_info->num_images - 1; i++) {
- memcpy(&wim_info->images[i], &wim_info->images[i + 1],
- sizeof(struct image_info));
- wim_info->images[i].index--;
- }
+ memcpy(&wim_info->images[image - 1],
+ &wim_info->images[image],
+ (wim_info->num_images - image) * sizeof(struct image_info));
if (--wim_info->num_images == 0) {
free_wim_info(wim_info);
*wim_info_p = NULL;
+ } else {
+ for (int i = image - 1; i < wim_info->num_images; i++)
+ wim_info->images[i].index--;
}
}
size_t xml_get_max_image_name_len(const WIMStruct *w)
{
- size_t len = 0;
- uint i;
- uint num_images = w->wim_info->num_images;
- for (i = 0; i < num_images; i++)
- len = max(len, strlen(w->wim_info->images[i].name));
- return len;
+ size_t max_len = 0;
+ if (w->wim_info) {
+ size_t len;
+ for (int i = 0; i < w->wim_info->num_images; i++) {
+ len = strlen(w->wim_info->images[i].name);
+ if (len > max_len)
+ max_len = len;
+ }
+ }
+ return max_len;
}
#ifdef ENABLE_CUSTOM_MEMORY_ALLOCATOR
static int calculate_dentry_statistics(struct dentry *dentry, void *arg)
{
- struct image_info *info = arg;
+ struct image_info *info = arg;
struct lookup_table *lookup_table = info->lookup_table;
const struct inode *inode = dentry->d_inode;
struct lookup_table_entry *lte;
/* Update directory count and file count.
*
* Each dentry counts as either a file or a directory, but not both.
- * The root directory is an exception: it is not counted.
+ * The root directory is an exception: it is not counted at all.
*
* Symbolic links and junction points (and presumably other reparse
* points) count as regular files. This is despite the fact that
else
info->file_count++;
- /*
+ /*
* Update total bytes and hard link bytes.
*
* Unfortunately there are some inconsistencies/bugs in the way this is
return 0;
}
+/*
+ * Calculate what to put in the <FILECOUNT>, <DIRCOUNT>, <TOTALBYTES>, and
+ * <HARDLINKBYTES> elements of each <IMAGE>.
+ *
+ * Please note there is no official documentation for exactly how this is done.
+ * But, see calculate_dentry_statistics().
+ */
void xml_update_image_info(WIMStruct *w, int image)
{
struct image_info *image_info;
- struct dentry *root;
char *flags_save;
DEBUG("Updating the image info for image %d", image);
flags_save = image_info->flags;
image_info->lookup_table = w->lookup_table;
-
for_dentry_in_tree(w->image_metadata[image - 1].root_dentry,
calculate_dentry_statistics,
image_info);
-
- image_info->lookup_table = NULL;
image_info->flags = flags_save;
image_info->last_modification_time = get_wim_timestamp();
}
struct wim_info *wim_info;
struct image_info *image_info;
- wimlib_assert(name);
-
- DEBUG("Adding image: name = %s", name);
+ wimlib_assert(name != NULL);
/* If this is the first image, allocate the struct wim_info. Otherwise
* use the existing struct wim_info. */
if (w->wim_info) {
wim_info = w->wim_info;
} else {
- DEBUG("Allocing struct wim_info with 1 image");
wim_info = CALLOC(1, sizeof(struct wim_info));
- if (!wim_info) {
- ERROR("Could not allocate WIM information struct--- "
- "out of memory");
+ if (!wim_info)
return WIMLIB_ERR_NOMEM;
- }
}
image_info = add_image_info_struct(wim_info);
out_destroy_image_info:
destroy_image_info(image_info);
+ wim_info->num_images--;
out_free_wim_info:
- if (w->wim_info)
- wim_info->num_images--;
- else
+ if (wim_info != w->wim_info)
FREE(wim_info);
- ERROR("Out of memory");
return WIMLIB_ERR_NOMEM;
}
-/* Prints information about the specified image from struct wim_info structure.
- * @image may be WIM_ALL_IMAGES. */
+/* Prints information about the specified image from struct wim_info structure.
+ * */
void print_image_info(const struct wim_info *wim_info, int image)
{
- uint i;
const struct image_info *image_info;
const char *desc;
+ char buf[50];
+ wimlib_assert(image >= 1 && image <= wim_info->num_images);
- if (image == WIM_ALL_IMAGES) {
- for (i = 1; i <= wim_info->num_images; i++)
- print_image_info(wim_info, i);
- } else {
- time_t time;
- char *p;
-
- image_info = &wim_info->images[image - 1];
-
- printf("Index: %"PRIu64"\n",
- image_info->index);
- printf("Name: %s\n",
- image_info->name);
-
- /* Always print the Description: part even if there is no
- * description. */
- if (image_info->description)
- desc = image_info->description;
- else
- desc = "";
- printf("Description: %s\n", desc);
-
- if (image_info->display_name)
- printf("Display Name: %s\n",
- image_info->display_name);
-
- if (image_info->display_description)
- printf("Display Description: %s\n",
- image_info->display_description);
-
- printf("Directory Count: %"PRIu64"\n",
- image_info->dir_count);
- printf("File Count: %"PRIu64"\n",
- image_info->file_count);
- printf("Total Bytes: %"PRIu64"\n",
- image_info->total_bytes);
- printf("Hard Link Bytes: %"PRIu64"\n",
- image_info->hard_link_bytes);
-
- time = wim_timestamp_to_unix(image_info->creation_time);
- p = asctime(gmtime(&time));
- *(strrchr(p, '\n')) = '\0';
-
- printf("Creation Time: %s UTC\n", p);
-
- time = wim_timestamp_to_unix(image_info->last_modification_time);
- p = asctime(gmtime(&time));
- *(strrchr(p, '\n')) = '\0';
-
- printf("Last Modification Time: %s UTC\n", p);
- if (image_info->windows_info_exists)
- print_windows_info(&image_info->windows_info);
- if (image_info->flags)
- printf("Flags: %s\n", image_info->flags);
- putchar('\n');
- }
+ image_info = &wim_info->images[image - 1];
+
+ printf("Index: %d\n", image_info->index);
+ printf("Name: %s\n", image_info->name);
+
+ /* Always print the Description: part even if there is no
+ * description. */
+ if (image_info->description)
+ desc = image_info->description;
+ else
+ desc = "";
+ printf("Description: %s\n", desc);
+
+ if (image_info->display_name)
+ printf("Display Name: %s\n",
+ image_info->display_name);
+
+ if (image_info->display_description)
+ printf("Display Description: %s\n",
+ image_info->display_description);
+
+ printf("Directory Count: %"PRIu64"\n", image_info->dir_count);
+ printf("File Count: %"PRIu64"\n", image_info->file_count);
+ printf("Total Bytes: %"PRIu64"\n", image_info->total_bytes);
+ printf("Hard Link Bytes: %"PRIu64"\n", image_info->hard_link_bytes);
+
+ wim_timestamp_to_str(image_info->creation_time, buf, sizeof(buf));
+ printf("Creation Time: %s\n", buf);
+
+ wim_timestamp_to_str(image_info->creation_time, buf, sizeof(buf));
+ printf("Last Modification Time: %s\n", buf);
+ if (image_info->windows_info_exists)
+ print_windows_info(&image_info->windows_info);
+ if (image_info->flags)
+ printf("Flags: %s\n", image_info->flags);
+ putchar('\n');
}
-/*
+/*
* Reads the XML data from a WIM file.
*/
-int read_xml_data(FILE *fp, const struct resource_entry *res, u8 **xml_data_ret,
- struct wim_info **info_ret)
+int read_xml_data(FILE *fp, const struct resource_entry *res_entry,
+ u8 **xml_data_ret, struct wim_info **info_ret)
{
u8 *xml_data;
xmlDoc *doc;
xmlNode *root;
int ret;
- DEBUG("XML data is %"PRIu64" bytes at offset %"PRIu64"",
- (u64)res->size, res->offset);
+ DEBUG("XML data is %"PRIu64" bytes at offset %"PRIu64"",
+ (u64)res_entry->size, res_entry->offset);
- if (resource_is_compressed(res)) {
+ if (resource_is_compressed(res_entry)) {
ERROR("XML data is supposed to be uncompressed");
ret = WIMLIB_ERR_XML;
goto out_cleanup_parser;
}
- if (res->size < 2) {
- ERROR("XML data must be at least 2 bytes");
+
+ if (res_entry->size < 2) {
+ ERROR("XML data must be at least 2 bytes long");
ret = WIMLIB_ERR_XML;
goto out_cleanup_parser;
}
- xml_data = MALLOC(res->size + 2);
+ xml_data = MALLOC(res_entry->size + 2);
if (!xml_data) {
ret = WIMLIB_ERR_NOMEM;
goto out_cleanup_parser;
}
- ret = read_uncompressed_resource(fp, res->offset, res->size, xml_data);
+
+ ret = read_uncompressed_resource(fp, res_entry->offset,
+ res_entry->size, xml_data);
if (ret != 0)
goto out_free_xml_data;
- xml_data[res->size] = 0;
- xml_data[res->size + 1] = 0;
+ /* Null-terminate just in case */
+ xml_data[res_entry->size] = 0;
+ xml_data[res_entry->size + 1] = 0;
- DEBUG("Parsing XML using libxml2 to create XML tree.");
-
- doc = xmlReadMemory(xml_data, res->size, "noname.xml", "UTF-16", 0);
+ DEBUG("Parsing XML using libxml2 to create XML tree");
+ doc = xmlReadMemory(xml_data, res_entry->size,
+ "noname.xml", "UTF-16", 0);
if (!doc) {
ERROR("Failed to parse XML data");
root = xmlDocGetRootElement(doc);
if (!root) {
- ERROR("Empty XML document");
+ ERROR("WIM XML data is an empty XML document");
ret = WIMLIB_ERR_XML;
goto out_free_doc;
}
DEBUG("Freeing XML tree.");
- xmlFreeDoc(doc);
- xmlCleanupParser();
*xml_data_ret = xml_data;
- return 0;
+ xml_data = NULL;
out_free_doc:
xmlFreeDoc(doc);
out_free_xml_data:
#define CHECK_RET ({ if (ret < 0) { \
ERROR("Error writing XML data"); \
ret = WIMLIB_ERR_WRITE; \
- goto err2; \
+ goto out_free_text_writer; \
} })
-/*
+/*
* Writes XML data to a WIM file.
*
* If @total_bytes is non-zero, it specifies what to write to the TOTALBYTES
* element in the XML data. If zero, TOTALBYTES is given the default value of
* the offset of the XML data.
*/
-int write_xml_data(const struct wim_info *wim_info, int image, FILE *out,
- u64 total_bytes)
+int write_xml_data(const struct wim_info *wim_info, int image, FILE *out,
+ u64 total_bytes, struct resource_entry *out_res_entry)
{
- xmlBuffer *buf;
+ xmlCharEncodingHandler *encoding_handler;
+ xmlOutputBuffer *out_buffer;
xmlTextWriter *writer;
- char *utf16_str;
int ret;
- int num_images;
- int i;
- const xmlChar *content;
- size_t len;
- size_t utf16_len;
- size_t bytes_written;
+ off_t start_offset;
+ off_t end_offset;
- wimlib_assert(image == WIM_ALL_IMAGES ||
- (wim_info != NULL && image >= 1 &&
+ wimlib_assert(image == WIMLIB_ALL_IMAGES ||
+ (wim_info != NULL && image >= 1 &&
image <= wim_info->num_images));
+ start_offset = ftello(out);
+ if (start_offset == -1)
+ return WIMLIB_ERR_WRITE;
+
+ DEBUG("Writing XML data for image %d at offset %"PRIu64,
+ image, start_offset);
+
+ /* 2 bytes endianness marker for UTF-16LE. This is _required_ for WIM
+ * XML data. */
+ if ((putc(0xff, out)) == EOF || (putc(0xfe, out) == EOF)) {
+ ERROR_WITH_ERRNO("Error writing XML data");
+ return WIMLIB_ERR_WRITE;
+ }
+
/* The contents of the <TOTALBYTES> element in the XML data, under the
- * <WIM> element not the <IMAGE> element, is (for non-spit WIMs) the
- * size of the WIM file excluding the XML data and integrity table,
- * which is the current offset, since the XML data goes at the end of
- * the WIM file before the integrity table. */
- if (total_bytes == 0) {
- total_bytes = ftello(out);
- if (total_bytes == (u64)-1)
- return WIMLIB_ERR_WRITE;
+ * <WIM> element (not the <IMAGE> element), is for non-split WIMs the
+ * size of the WIM file excluding the XML data and integrity table.
+ * This should be equal to the current position in the output stream,
+ * since the XML data and integrity table are the last elements of the
+ * WIM.
+ *
+ * For split WIMs, <TOTALBYTES> takes into account the entire WIM, not
+ * just the current part. In that case, @total_bytes should be passed
+ * in to this function. */
+ if (total_bytes == 0)
+ total_bytes = start_offset;
+
+ xmlInitCharEncodingHandlers();
+
+ /* The encoding of the XML data must be UTF-16LE. */
+ encoding_handler = xmlGetCharEncodingHandler(XML_CHAR_ENCODING_UTF16LE);
+ if (!encoding_handler) {
+ ERROR("Failed to get XML character encoding handler for UTF-16LE");
+ ret = WIMLIB_ERR_LIBXML_UTF16_HANDLER_NOT_AVAILABLE;
+ goto out_cleanup_char_encoding_handlers;
}
- DEBUG("Creating XML buffer and text writer.");
- buf = xmlBufferCreate();
- if (!buf) {
- ERROR("Failed to allocate XML buffer");
+ out_buffer = xmlOutputBufferCreateFile(out, encoding_handler);
+ if (!out_buffer) {
+ ERROR("Failed to allocate xmlOutputBuffer");
ret = WIMLIB_ERR_NOMEM;
- goto err0;
+ goto out_cleanup_char_encoding_handlers;
}
- writer = xmlNewTextWriterMemory(buf, 0);
+
+ writer = xmlNewTextWriter(out_buffer);
if (!writer) {
- ERROR("Failed to allocate XML writer");
+ ERROR("Failed to allocate xmlTextWriter");
ret = WIMLIB_ERR_NOMEM;
- goto err1;
+ goto out_output_buffer_close;
}
- /* XXX */
- /* M$'s WIM files do not have XML declarations, so do not write one.
- * I'm not sure how we can force the document to be written in UTF-16
- * without calling xmlTextWriterStartDocument(), though, so currently it
- * is composed in a buffer UTF-8, then converted to UTF-16. */
-#if 0
- ret = xmlTextWriterStartDocument(writer, NULL, "UTF-16", NULL);
- CHECK_RET;
-#endif
-
DEBUG("Writing <WIM> element");
+
ret = xmlTextWriterStartElement(writer, "WIM");
CHECK_RET;
total_bytes);
CHECK_RET;
- if (wim_info)
- num_images = wim_info->num_images;
- else
- num_images = 0;
- DEBUG("Writing %u <IMAGE> elements", num_images);
-
- for (i = 1; i <= num_images; i++) {
- if (image != WIM_ALL_IMAGES && i != image)
- continue;
- DEBUG("Writing <IMAGE> element for image %d", i);
- ret = xml_write_image_info(writer, &wim_info->images[i - 1]);
- CHECK_RET;
+ if (wim_info != NULL) {
+ int first, last;
+ if (image == WIMLIB_ALL_IMAGES) {
+ first = 1;
+ last = wim_info->num_images;
+ } else {
+ first = image;
+ last = image;
+ }
+ DEBUG("Writing %d <IMAGE> elements", last - first + 1);
+ for (int i = first; i <= last; i++) {
+ ret = xml_write_image_info(writer, &wim_info->images[i - 1]);
+ CHECK_RET;
+ }
}
ret = xmlTextWriterEndElement(writer);
ret = xmlTextWriterEndDocument(writer);
CHECK_RET;
- DEBUG("Done composing XML document. Now converting to UTF-16 and "
- "writing it to the output file.");
-
- content = xmlBufferContent(buf);
- len = xmlBufferLength(buf);
+ DEBUG("Ended XML document");
- utf16_str = utf8_to_utf16(content, len, &utf16_len);
- if (!utf16_str) {
- ret = WIMLIB_ERR_NOMEM;
- goto err2;
- }
+ /* Call xmlFreeTextWriter() before ftello() because the former will
+ * flush the file stream. */
+ xmlFreeTextWriter(writer);
+ writer = NULL;
- if ((putc(0xff, out)) == EOF || (putc(0xfe, out) == EOF) ||
- ((bytes_written = fwrite(utf16_str, 1, utf16_len, out))
- != utf16_len)) {
- ERROR_WITH_ERRNO("Error writing XML data");
+ end_offset = ftello(out);
+ if (end_offset == -1) {
ret = WIMLIB_ERR_WRITE;
- goto err3;
- }
-
- DEBUG("Cleaning up.");
-
- ret = 0;
-err3:
- FREE(utf16_str);
-err2:
+ } else {
+ ret = 0;
+ out_res_entry->offset = start_offset;
+ out_res_entry->size = end_offset - start_offset;
+ out_res_entry->original_size = end_offset - start_offset;
+ out_res_entry->flags = WIM_RESHDR_FLAG_METADATA;
+ }
+out_free_text_writer:
+ /* xmlFreeTextWriter will free the attached xmlOutputBuffer. */
xmlFreeTextWriter(writer);
-err1:
- xmlBufferFree(buf);
-err0:
+ out_buffer = NULL;
+out_output_buffer_close:
+ if (out_buffer != NULL)
+ xmlOutputBufferClose(out_buffer);
+out_cleanup_char_encoding_handlers:
+ xmlCleanupCharEncodingHandlers();
+out:
+ if (ret == 0)
+ DEBUG("Successfully wrote XML data");
return ret;
}
/* Returns the name of the specified image. */
WIMLIBAPI const char *wimlib_get_image_name(const WIMStruct *w, int image)
{
- DEBUG("Getting the name of image %d", image);
if (image < 1 || image > w->hdr.image_count)
return NULL;
-
return w->wim_info->images[image - 1].name;
}
/* Returns the description of the specified image. */
-WIMLIBAPI const char *wimlib_get_image_description(const WIMStruct *w,
+WIMLIBAPI const char *wimlib_get_image_description(const WIMStruct *w,
int image)
{
- DEBUG("Getting the description of image %d", image);
if (image < 1 || image > w->hdr.image_count)
return NULL;
-
return w->wim_info->images[image - 1].description;
}
/* Determines if an image name is already used by some image in the WIM. */
WIMLIBAPI bool wimlib_image_name_in_use(const WIMStruct *w, const char *name)
{
- int i;
-
- DEBUG("Checking to see if the image name `%s' is already in use", name);
- if (!name || !w->wim_info)
+ if (!name || !*name)
return false;
- for (i = 1; i <= w->wim_info->num_images; i++)
+ for (int i = 1; i <= w->hdr.image_count; i++)
if (strcmp(w->wim_info->images[i - 1].name, name) == 0)
return true;
-
return false;
}
+/* Extracts the raw XML data to a file stream. */
WIMLIBAPI int wimlib_extract_xml_data(WIMStruct *w, FILE *fp)
{
- DEBUG("Extracting the XML data.");
- if (fwrite(w->xml_data, 1, w->hdr.xml_res_entry.size, fp) !=
+ if (!w->xml_data)
+ return WIMLIB_ERR_INVALID_PARAM;
+ if (fwrite(w->xml_data, 1, w->hdr.xml_res_entry.size, fp) !=
w->hdr.xml_res_entry.size) {
ERROR_WITH_ERRNO("Failed to extract XML data");
return WIMLIB_ERR_WRITE;
DEBUG("Setting the name of image %d to %s", image, name);
- if (!w)
- return WIMLIB_ERR_INVALID_PARAM;
-
if (!name || !*name) {
ERROR("Must specify a non-empty string for the image name");
return WIMLIB_ERR_INVALID_PARAM;
}
+
if (image < 1 || image > w->hdr.image_count) {
ERROR("%d is not a valid image", image);
return WIMLIB_ERR_INVALID_IMAGE;
}
p = STRDUP(name);
- if (!p) {
- ERROR("Out of memory");
+ if (!p)
return WIMLIB_ERR_NOMEM;
- }
+
FREE(w->wim_info->images[image - 1].name);
w->wim_info->images[image - 1].name = p;
return 0;
}
/* Sets the description of an image in the WIM. */
-WIMLIBAPI int wimlib_set_image_descripton(WIMStruct *w, int image,
+WIMLIBAPI int wimlib_set_image_descripton(WIMStruct *w, int image,
const char *description)
{
char *p;
- DEBUG("Setting the description of image %d to %s", image, description);
-
- if (!w)
- return WIMLIB_ERR_INVALID_PARAM;
-
if (image < 1 || image > w->hdr.image_count) {
ERROR("%d is not a valid image", image);
return WIMLIB_ERR_INVALID_IMAGE;
}
if (description) {
p = STRDUP(description);
- if (!p) {
- ERROR("Out of memory");
+ if (!p)
return WIMLIB_ERR_NOMEM;
- }
} else {
p = NULL;
}
return 0;
}
+/* Set the <FLAGS> element of a WIM image */
WIMLIBAPI int wimlib_set_image_flags(WIMStruct *w, int image,
const char *flags)
{
char *p;
- DEBUG("Setting the flags of image %d to %s", image, flags);
-
if (image < 1 || image > w->hdr.image_count) {
ERROR("%d is not a valid image", image);
return WIMLIB_ERR_INVALID_IMAGE;
}
if (flags) {
p = STRDUP(flags);
- if (!p) {
- ERROR("Out of memory");
+ if (!p)
return WIMLIB_ERR_NOMEM;
- }
} else {
p = NULL;
}