zdoom/gzdoom-gles
0
0
Fork 0
mirror of https://github.com/ZDoom/gzdoom-gles.git synced 2025-01-18 22:51:39 +00:00
Code Issues Projects Releases Packages Wiki Activity

- fixed: DoSetMapSection could cause a stack overflow on large maps. Made it iterative instead of recursive to avoid that.

Browse source
This commit is contained in:
Christoph Oelckers 2016-07-27 16:27:40 +02:00
parent c9f93d9c88
commit da1762ac2c
1 changed files with 17 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
src/gl/data/gl_setup.cpp
View file

@ -69,26 +69,34 @@ void InitGLRMapinfoData();
// //
// //
//========================================================================== //==========================================================================
static TArray<subsector_t *> MapSectionCollector;
static void DoSetMapSection(subsector_t *sub, int num) static void DoSetMapSection(subsector_t *sub, int num)
{ {
MapSectionCollector.Resize(1);
MapSectionCollector[0] = sub;
sub->mapsection = num; sub->mapsection = num;
for (unsigned a = 0; a < MapSectionCollector.Size(); a++)
for(DWORD i=0;i<sub->numlines;i++)
{ {
seg_t * seg = sub->firstline + i; sub = MapSectionCollector[a];
for (DWORD i = 0; i < sub->numlines; i++)
if (seg->PartnerSeg)
{ {
subsector_t * sub2 = seg->PartnerSeg->Subsector; seg_t * seg = sub->firstline + i;
if (sub2->mapsection != num) if (seg->PartnerSeg)
{ {
assert(sub2->mapsection == 0); subsector_t * sub2 = seg->PartnerSeg->Subsector;
DoSetMapSection(sub2, num);
if (sub2->mapsection != num)
{
assert(sub2->mapsection == 0);
sub2->mapsection = num;
MapSectionCollector.Push(sub2);
}
} }
} }
} }
MapSectionCollector.Clear();
} }
//========================================================================== //==========================================================================