X-Git-Url: https://wimlib.net/git/?a=blobdiff_plain;f=src%2Ftagged_items.c;h=e3ba53f496612c7d95e8dabaa12d3512865edca6;hb=9285ed562c208959e0d5034e0ab4289cc40a31a2;hp=3a12974ae5b6b8ad3ae95efe375c91160ae8db37;hpb=27b30056e4520e9b5b9d0846f438311746345f83;p=wimlib

diff --git a/src/tagged_items.c b/src/tagged_items.c
index 3a12974a..e3ba53f4 100644
--- a/src/tagged_items.c
+++ b/src/tagged_items.c
@@ -8,20 +8,18 @@
 /*
  * Copyright (C) 2014 Eric Biggers
  *
- * This file is part of wimlib, a library for working with WIM files.
+ * 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
+ * Software Foundation; either version 3 of the License, or (at your option) any
+ * later version.
  *
- * wimlib is free software; you can redistribute it and/or modify it under the
- * terms of the GNU General Public License as published by the Free
- * Software Foundation; either version 3 of the License, or (at your option)
- * any later version.
- *
- * wimlib is distributed in the hope that it will be useful, but WITHOUT ANY
- * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
- * A PARTICULAR PURPOSE. See the GNU General Public License for more
+ * This file is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License for more
  * details.
  *
- * You should have received a copy of the GNU General Public License
- * along with wimlib; if not, see http://www.gnu.org/licenses/.
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this file; if not, see http://www.gnu.org/licenses/.
  */
 
 #ifdef HAVE_CONFIG_H
@@ -45,7 +43,7 @@ struct tagged_item_header {
 	/* Unique identifier for this item.  */
 	le32 tag;
 
-	/* Size of the data of this tagged item, in bytes.  This includes this
+	/* Size of the data of this tagged item, in bytes.  This excludes this
 	 * header and should be a multiple of 8.
 	 *
 	 * (Actually, the MS implementation seems to round this up to an 8 byte
@@ -69,7 +67,7 @@ struct wimlib_unix_data_disk {
 	le32 uid;
 	le32 gid;
 	le32 mode;
-	le32 reserved;
+	le32 rdev;
 };
 
 /* Retrieves the first tagged item with the specified tag and minimum length
@@ -98,10 +96,10 @@ inode_get_tagged_item(const struct wim_inode *inode,
 			return hdr->data;
 
 		len = (len + 7) & ~7;
-		if (len >= len_remaining)
+		if (len_remaining <= sizeof(struct tagged_item_header) + len)
 			return NULL;
-		len_remaining -= len;
-		p = hdr->data + len;
+		len_remaining -= sizeof(struct tagged_item_header) + len;
+		p += sizeof(struct tagged_item_header) + len;
 	}
 	return NULL;
 }
@@ -118,7 +116,7 @@ inode_add_tagged_item(struct wim_inode *inode, u32 tag, u32 len)
 
 	/* We prepend the item instead of appending it because it's easier.  */
 
-	itemsize = sizeof(struct tagged_item_header) + len;
+	itemsize = sizeof(struct tagged_item_header) + ((len + 7) & ~7);
 	newsize = itemsize + inode->i_extra_size;
 
 	buf = MALLOC(newsize);
@@ -135,7 +133,7 @@ inode_add_tagged_item(struct wim_inode *inode, u32 tag, u32 len)
 	hdr = (struct tagged_item_header *)buf;
 	hdr->tag = cpu_to_le32(tag);
 	hdr->length = cpu_to_le32(len);
-	return hdr->data;
+	return memset(hdr->data, 0, (len + 7) & ~7);
 }
 
 static inline struct wimlib_unix_data_disk *
@@ -178,19 +176,20 @@ inode_get_unix_data(const struct wim_inode *inode,
 	unix_data->uid = le32_to_cpu(p->uid);
 	unix_data->gid = le32_to_cpu(p->gid);
 	unix_data->mode = le32_to_cpu(p->mode);
+	unix_data->rdev = le32_to_cpu(p->rdev);
 	return true;
 }
 
 /* Sets UNIX data on the specified WIM inode.
  * This is a wimlib extension.
  *
- * Callers must specify all of @uid, @gid, and @mode.  If the inode does not yet
+ * Callers must specify all members in @unix_data.  If the inode does not yet
  * have UNIX data, it is given these values.  Otherwise, only the values that
  * also have the corresponding flags in @which set are changed.
  *
  * Returns %true if successful, %false if failed (out of memory).  */
 bool
-inode_set_unix_data(struct wim_inode *inode, u32 uid, u32 gid, u32 mode,
+inode_set_unix_data(struct wim_inode *inode, struct wimlib_unix_data *unix_data,
 		    int which)
 {
 	struct wimlib_unix_data_disk *p;
@@ -200,14 +199,15 @@ inode_set_unix_data(struct wim_inode *inode, u32 uid, u32 gid, u32 mode,
 		p = inode_add_unix_data_disk(inode);
 		if (!p)
 			return false;
-		p->reserved = cpu_to_le32(0);
-		which = UNIX_DATA_UID | UNIX_DATA_GID | UNIX_DATA_MODE;
+		which = UNIX_DATA_ALL;
 	}
 	if (which & UNIX_DATA_UID)
-		p->uid = cpu_to_le32(uid);
+		p->uid = cpu_to_le32(unix_data->uid);
 	if (which & UNIX_DATA_GID)
-		p->gid = cpu_to_le32(gid);
+		p->gid = cpu_to_le32(unix_data->gid);
 	if (which & UNIX_DATA_MODE)
-		p->mode = cpu_to_le32(mode);
+		p->mode = cpu_to_le32(unix_data->mode);
+	if (which & UNIX_DATA_RDEV)
+		p->rdev = cpu_to_le32(unix_data->rdev);
 	return true;
 }