[OMPT-svn-commit] OpenMPT r20597 - trunk/OpenMPT/soundlib

[<sv...@op...>]

Author: sagamusix
Date: Sun Apr 14 16:15:51 2024
New Revision: 20597
URL: https://source.openmpt.org/browse/openmpt/?op=revision&rev=20597

Log:
[Fix] MED: Possible crash when trying to access non-existing pattern.

Modified:
   trunk/OpenMPT/soundlib/Load_med.cpp
   trunk/OpenMPT/soundlib/ModSequence.cpp
   trunk/OpenMPT/soundlib/ModSequence.h

Modified: trunk/OpenMPT/soundlib/Load_med.cpp
==============================================================================
--- trunk/OpenMPT/soundlib/Load_med.cpp	Sun Apr 14 14:58:52 2024	(r20596)
+++ trunk/OpenMPT/soundlib/Load_med.cpp	Sun Apr 14 16:15:51 2024	(r20597)
@@ -1627,7 +1627,7 @@
 
 		if(numSongs > 1)
 		{
-			PATTERNINDEX firstPat = order.EnsureUnique(0);
+			PATTERNINDEX firstPat = order.EnsureUnique(order.GetFirstValidIndex());
 			if(firstPat != PATTERNINDEX_INVALID)
 			{
 				for(CHANNELINDEX chn = 0; chn < m_nChannels; chn++)

Modified: trunk/OpenMPT/soundlib/ModSequence.cpp
==============================================================================
--- trunk/OpenMPT/soundlib/ModSequence.cpp	Sun Apr 14 14:58:52 2024	(r20596)
+++ trunk/OpenMPT/soundlib/ModSequence.cpp	Sun Apr 14 16:15:51 2024	(r20597)
@@ -149,6 +149,18 @@
 }
 
 
+ORDERINDEX ModSequence::GetFirstValidIndex() const noexcept
+{
+	const ORDERINDEX length = GetLength();
+	for(ORDERINDEX ord = 0; ord < length; ord++)
+	{
+		if(IsValidPat(ord))
+			return ord;
+	}
+	return ORDERINDEX_INVALID;
+}
+
+
 void ModSequence::Remove(ORDERINDEX posBegin, ORDERINDEX posEnd) noexcept
 {
 	if(posEnd < posBegin || posEnd >= size())

Modified: trunk/OpenMPT/soundlib/ModSequence.h
==============================================================================
--- trunk/OpenMPT/soundlib/ModSequence.h	Sun Apr 14 14:58:52 2024	(r20596)
+++ trunk/OpenMPT/soundlib/ModSequence.h	Sun Apr 14 16:15:51 2024	(r20597)
@@ -98,6 +98,9 @@
 	ORDERINDEX GetPreviousOrderIgnoringSkips(const ORDERINDEX start) const noexcept;
 	ORDERINDEX GetNextOrderIgnoringSkips(const ORDERINDEX start) const noexcept;
 
+	// Returns the first item that contains a pattern that actually exists, ORDERINDEX_INVALID if no such item exists.
+	ORDERINDEX GetFirstValidIndex() const noexcept;
+
 	// Find an order item that contains a given pattern number.
 	ORDERINDEX FindOrder(PATTERNINDEX pat, ORDERINDEX startSearchAt = 0, bool searchForward = true) const noexcept;