phaneron/Kelsidavis-WoWee
1
0
Fork 0
mirror of https://github.com/Kelsidavis/WoWee.git synced 2026-05-07 09:33:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

fix(wob): cap material count at 256 on save (matches load limit)

Browse source 
Save previously wrote raw materials.size() as the count, then iterated
all materials. Load caps at 256, so a build with >256 materials would
write fine but truncate on round-trip and the post-truncation block
would be misread as the next group's data. Cap at save and only write
the first 256.
This commit is contained in:
Kelsi 2026-05-06 09:04:18 -07:00
parent 4d6c65ab73
commit 4a534c24e8
1 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/pipeline/wowee_building.cpp
View file

@ -279,10 +279,14 @@ bool WoweeBuildingLoader::save(const WoweeBuilding& bld, const std::string& base
for (const auto& tp : grp.texturePaths) writeStr(tp);
// Write material data
uint32_t mc = static_cast<uint32_t>(grp.materials.size());
// Write material data — cap at 256 to match load-side limit so a
// pathological in-memory count can't write a file the loader will
// reject and produce a partially-zero build on round-trip.
uint32_t mc = static_cast<uint32_t>(
std::min<size_t>(grp.materials.size(), 256));
f.write(reinterpret_cast<const char*>(&mc), 4);
for (const auto& mat : grp.materials) {
for (uint32_t mi = 0; mi < mc; mi++) {
const auto& mat = grp.materials[mi];
writeStr(mat.texturePath);
f.write(reinterpret_cast<const char*>(&mat.flags), 4);
f.write(reinterpret_cast<const char*>(&mat.shader), 4);