- fixed validation issue with maps that have a wall with nextwall == …

…-1 and nextsector > -1. In this case the bogus nextsector was not reset.
ZDoom · Feb 15, 2022 · de1cbb9 · de1cbb9
1 parent 21fddd6
commit de1cbb9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/source/core/gamefuncs.h b/source/core/gamefuncs.h
@@ -139,9 +139,8 @@ void vertexscan(walltype* startwall, func mark)
 	while (true)
 	{
 		auto thelastwall = wal->lastWall();
-		// thelastwall can be null here if the map is bogus.
-		if (!thelastwall || !thelastwall->twoSided()) break;
-
+		// thelastwall can be null here if the map is bogus. 
+		if (!thelastwall || thelastwall->nextwall < 0) break;
 		wal = thelastwall->nextWall();
 		if (walbitmap.Check(wall.IndexOf(wal))) break;
 		mark(wal);

diff --git a/source/core/maploader.cpp b/source/core/maploader.cpp
@@ -831,7 +831,11 @@ void setWallSectors()
 				wal.nextsector = wal.nextWall()->sector;
 			}
 		}
-		else wal.nextwall = -1;
+		else
+		{
+			wal.nextwall = -1;
+			wal.nextsector = -1;
+		}
 	}
 
 }