phaneron/Kelsidavis-WoWee
1
0
Fork 0
mirror of https://github.com/Kelsidavis/WoWee.git synced 2026-05-06 17:13:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

fix(wot): cap doodad/WMO placement count at 100k on load

Browse source 
Same defense pattern as the editor JSON loaders. Real ADTs cap at
~64k MDDF entries and ~5k in practice; 100k matches the editor
ObjectPlacer cap so an extreme WOT can't bloat the in-memory
terrain past what the editor itself would accept.
This commit is contained in:
Kelsi 2026-05-06 10:03:08 -07:00
parent fc284c7460
commit e7462efaf6
1 changed files with 6 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/pipeline/wowee_terrain_loader.cpp
View file

@ -172,8 +172,13 @@ bool WoweeTerrainLoader::loadMetadata(const std::string& wotPath, ADTTerrain& te
if (!std::isfinite(b)) b = 0.0f;
if (!std::isfinite(c)) c = 0.0f;
};
// Caps match the editor-side ObjectPlacer (100k for objects). Per-
// tile, real ADTs cap at ~64k MDDF entries (uint32 indices) and
// far fewer in practice (~5k for dense zones).
constexpr size_t kMaxPlacements = 100'000;
if (j.contains("doodads") && j["doodads"].is_array()) {
for (const auto& jd : j["doodads"]) {
if (terrain.doodadPlacements.size() >= kMaxPlacements) break;
ADTTerrain::DoodadPlacement dp{};
dp.nameId = jd.value("nameId", 0u);
dp.uniqueId = jd.value("uniqueId", 0u);
@ -198,6 +203,7 @@ bool WoweeTerrainLoader::loadMetadata(const std::string& wotPath, ADTTerrain& te
}
if (j.contains("wmos") && j["wmos"].is_array()) {
for (const auto& jw : j["wmos"]) {
if (terrain.wmoPlacements.size() >= kMaxPlacements) break;
ADTTerrain::WMOPlacement wp{};
wp.nameId = jw.value("nameId", 0u);
wp.uniqueId = jw.value("uniqueId", 0u);