goto out_of_memory;
sd_disk = (const struct wim_security_data_disk *)buf;
- sd->total_length = le32_to_cpu(sd_disk->total_length);
+ sd->total_length = ALIGN(le32_to_cpu(sd_disk->total_length), 8);
sd->num_entries = le32_to_cpu(sd_disk->num_entries);
- DEBUG("Reading security data: num_entries=%u, total_length=%u",
- sd->num_entries, sd->total_length);
-
/* Length field of 0 is a special case that really means length
* of 8. */
if (sd->total_length == 0)
/* Return immediately if no security descriptors. */
if (sd->num_entries == 0)
- goto out_align_total_length;
+ goto out_descriptors_ready;
/* Allocate a new buffer for the sizes array */
sd->sizes = MALLOC(sizes_size);
goto out_of_memory;
p += sd->sizes[i];
}
-out_align_total_length:
- total_len = (total_len + 7) & ~7;
- sd->total_length = (sd->total_length + 7) & ~7;
- if (total_len != sd->total_length) {
- WARNING("Expected WIM security data total length of "
- "%u bytes, but calculated %u bytes",
- sd->total_length, (unsigned)total_len);
+out_descriptors_ready:
+ if (ALIGN(total_len, 8) != sd->total_length) {
+ WARNING("Stored WIM security data total length was "
+ "%"PRIu32" bytes, but calculated %"PRIu32" bytes",
+ sd->total_length, (u32)total_len);
}
*sd_ret = sd;
ret = 0;
write_wim_security_data(const struct wim_security_data * restrict sd,
u8 * restrict p)
{
- DEBUG("Writing security data (total_length = %"PRIu32", num_entries "
- "= %"PRIu32")", sd->total_length, sd->num_entries);
-
u8 *orig_p = p;
struct wim_security_data_disk *sd_disk = (struct wim_security_data_disk*)p;
u32 num_entries = sd->num_entries;
*p++ = 0;
wimlib_assert(p - orig_p == sd->total_length);
-
- DEBUG("Successfully wrote security data.");
return p;
}
sd->descriptors[sd->num_entries] = descr_copy;
sd->sizes[sd->num_entries] = size;
sd->num_entries++;
- DEBUG("There are now %u security descriptors", sd->num_entries);
bret = insert_sd_node(sd_set, new);
wimlib_assert(bret);
security_id = new->security_id;