GLK: Fix off-by-one memory read

The for loop shouldn't check previous entity on the first iteration
scummvm · Jul 7, 2019 · 1c87f42 · 1c87f42
1 parent da7ed2d
commit 1c87f42
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/engines/glk/alan2/alan2.cpp b/engines/glk/alan2/alan2.cpp
@@ -161,8 +161,11 @@ static void syncEventQueue(Common::Serializer &s) {
 	EvtqElem *arr = eventq;
 
 	if (s.isLoading()) {
-		for (i = 0; arr[i - 1].time != 0; ++i)
+		i = 0;
+		do {
 			arr[i].synchronize(s);
+			i++;
+		} while (arr[i - 1].time != 0);
 		etop = i - 1;
 	} else {
 		// Mark the top