From 30ea3bc9c25cd3b684a99d70d8fde629d54f0185 Mon Sep 17 00:00:00 2001 From: Chandler Cox Date: Mon, 13 Jan 2025 07:16:31 -0600 Subject: [PATCH] Add check for empty array and use `IsInRange` for version check. --- Source/Editor/Cooker/Steps/DeployDataStep.cpp | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/Source/Editor/Cooker/Steps/DeployDataStep.cpp b/Source/Editor/Cooker/Steps/DeployDataStep.cpp index 38a7bcbb6..489321d99 100644 --- a/Source/Editor/Cooker/Steps/DeployDataStep.cpp +++ b/Source/Editor/Cooker/Steps/DeployDataStep.cpp @@ -224,11 +224,14 @@ bool DeployDataStep::Perform(CookingData& data) { Array versionParts; pathParts[i - 1].Split('.', versionParts); - const String majorVersion = versionParts[0].TrimTrailing(); - uint32 versionNum; - StringUtils::Parse(*majorVersion, majorVersion.Length(), &versionNum); - if (versionNum < GAME_BUILD_DOTNET_RUNTIME_MAX_VER || versionNum > GAME_BUILD_DOTNET_RUNTIME_MIN_VER) // Check for major part - version = majorVersion; + if (!versionParts.IsEmpty()) + { + const String majorVersion = versionParts[0].TrimTrailing(); + int32 versionNum; + StringUtils::Parse(*majorVersion, majorVersion.Length(), &versionNum); + if (Math::IsInRange(versionNum, GAME_BUILD_DOTNET_RUNTIME_MIN_VER, GAME_BUILD_DOTNET_RUNTIME_MAX_VER)) // Check for major part + version = majorVersion; + } } }