From c9100aaceebc61a1ae3abf3c25aea158158e5f8f Mon Sep 17 00:00:00 2001
From: "Marco Antonio J. Costa" <marco.antonio.costa@gmail.com>
Date: Sun, 9 Aug 2026 12:01:24 -0300
Subject: [PATCH] return after asserting an illegal retreat direction

When no cardinal direction resolves, ubDirection stays 255 and the
tail feeds it to GetSectorMvtTimeForGroup, which reads
SectorInfo[...].ubTraversability[255] on a UINT8[5]. This runs in
release too, since that read is a plain call, not an assert. Bail out
instead.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 Strategic/Strategic Movement.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Strategic/Strategic Movement.cpp b/Strategic/Strategic Movement.cpp
index eec250229..ab1938fd0 100644
--- a/Strategic/Strategic Movement.cpp	
+++ b/Strategic/Strategic Movement.cpp	
@@ -4653,6 +4653,7 @@ void RetreatGroupToPreviousSector( GROUP *pGroup )
 
 		AssertMsg( 0, String("Player group attempting illegal retreat from %c%d to %c%d.",
 			pGroup->ubSectorY+'A'-1, pGroup->ubSectorX, pGroup->ubNextY+'A'-1, pGroup->ubNextX ) );
+		return;
 	}
 
 	//Calc time to get to next waypoint...