From 9e402ff91fb6a2a737042c71fb684e9dc3677180 Mon Sep 17 00:00:00 2001
From: Alex Evgrashin <aevgrashin@yandex.ru>
Date: Fri, 2 Feb 2024 12:59:44 +0100
Subject: [PATCH] Fixed out of bounds exception during resampling (#74)

* Fixed out of bounds resample

* Update test.yml

* Update test.yml

* Update test.yml
---
 .github/workflows/test.yml                                  | 4 +++-
 Packages/com.whisper.unity/Runtime/Utils/AudioUtils.cs      | 4 ++--
 Packages/com.whisper.unity/Tests/Runtime/AudioUtilsTests.cs | 4 ++--
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 78dc1bd..9b51129 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -8,7 +8,9 @@ on:
     branches:
       - master
 env:
-  UNITY_LICENSE: "<?xml version=\"1.0\" encoding=\"UTF-8\"?><root>\n    <License id=\"Terms\">\n        <MachineBindings>\n            <Binding Key=\"1\" Value=\"576562626572264761624c65526f7578\"/>\n            <Binding Key=\"2\" Value=\"576562626572264761624c65526f7578\"/>\n        </MachineBindings>\n        <MachineID Value=\"D7nTUnjNAmtsUMcnoyrqkgIbYdM=\"/>\n        <SerialHash Value=\"2033b8ac3e6faa3742ca9f0bfae44d18f2a96b80\"/>\n        <Features>\n            <Feature Value=\"33\"/>\n            <Feature Value=\"1\"/>\n            <Feature Value=\"12\"/>\n            <Feature Value=\"2\"/>\n            <Feature Value=\"24\"/>\n            <Feature Value=\"3\"/>\n            <Feature Value=\"36\"/>\n            <Feature Value=\"17\"/>\n            <Feature Value=\"19\"/>\n            <Feature Value=\"62\"/>\n        </Features>\n        <DeveloperData Value=\"AQAAAEY0LUJHUlgtWEQ0RS1aQ1dWLUM1SlctR0RIQg==\"/>\n        <SerialMasked Value=\"F4-BGRX-XD4E-ZCWV-C5JW-XXXX\"/>\n        <StartDate Value=\"2021-02-08T00:00:00\"/>\n        <UpdateDate Value=\"2021-02-09T00:34:57\"/>\n        <InitialActivationDate Value=\"2021-02-08T00:34:56\"/>\n        <LicenseVersion Value=\"6.x\"/>\n        <ClientProvidedVersion Value=\"2018.4.30f1\"/>\n        <AlwaysOnline Value=\"false\"/>\n        <Entitlements>\n            <Entitlement Ns=\"unity_editor\" Tag=\"UnityPersonal\" Type=\"EDITOR\" ValidTo=\"9999-12-31T00:00:00\"/>\n            <Entitlement Ns=\"unity_editor\" Tag=\"DarkSkin\" Type=\"EDITOR_FEATURE\" ValidTo=\"9999-12-31T00:00:00\"/>\n        </Entitlements>\n    </License>\n<Signature xmlns=\"http://www.w3.org/2000/09/xmldsig#\"><SignedInfo><CanonicalizationMethod Algorithm=\"http://www.w3.org/TR/2001/REC-xml-c14n-20010315#WithComments\"/><SignatureMethod Algorithm=\"http://www.w3.org/2000/09/xmldsig#rsa-sha1\"/><Reference URI=\"#Terms\"><Transforms><Transform Algorithm=\"http://www.w3.org/2000/09/xmldsig#enveloped-signature\"/></Transforms><DigestMethod Algorithm=\"http://www.w3.org/2000/09/xmldsig#sha1\"/><DigestValue>m0Db8UK+ktnOLJBtHybkfetpcKo=</DigestValue></Reference></SignedInfo><SignatureValue>o/pUbSQAukz7+ZYAWhnA0AJbIlyyCPL7bKVEM2lVqbrXt7cyey+umkCXamuOgsWPVUKBMkXtMH8L\n5etLmD0getWIhTGhzOnDCk+gtIPfL4jMo9tkEuOCROQAXCci23VFscKcrkB+3X6h4wEOtA2APhOY\nB+wvC794o8/82ffjP79aVAi57rp3Wmzx+9pe9yMwoJuljAy2sc2tIMgdQGWVmOGBpQm3JqsidyzI\nJWG2kjnc7pDXK9pwYzXoKiqUqqrut90d+kQqRyv7MSZXR50HFqD/LI69h68b7P8Bjo3bPXOhNXGR\n9YCoemH6EkfCJxp2gIjzjWW+l2Hj2EsFQi8YXw==</SignatureValue></Signature></root>"
+  UNITY_LICENSE: ${{ secrets.UNITY_LICENSE }}
+  UNITY_EMAIL: ${{ secrets.UNITY_EMAIL }}
+  UNITY_PASSWORD: ${{ secrets.UNITY_PASSWORD }}
 
 jobs:
   testAllModes:
diff --git a/Packages/com.whisper.unity/Runtime/Utils/AudioUtils.cs b/Packages/com.whisper.unity/Runtime/Utils/AudioUtils.cs
index 0871042..a84da39 100644
--- a/Packages/com.whisper.unity/Runtime/Utils/AudioUtils.cs
+++ b/Packages/com.whisper.unity/Runtime/Utils/AudioUtils.cs
@@ -56,8 +56,8 @@ public static float[] ChangeSampleRate(float[] src, int srcSampleRate, int dstSa
                 var low = Mathf.FloorToInt(index);
                 var dif = index - low;
 
-                if (low + 1 == srcLen)
-                    dst[i] = src[low];
+                 if (low + 1 >= srcLen)
+                    dst[i] = src[srcLen - 1];
                 else
                     dst[i] = Mathf.Lerp(src[low], src[low + 1], dif);
             }
diff --git a/Packages/com.whisper.unity/Tests/Runtime/AudioUtilsTests.cs b/Packages/com.whisper.unity/Tests/Runtime/AudioUtilsTests.cs
index 77281cc..5851bbe 100644
--- a/Packages/com.whisper.unity/Tests/Runtime/AudioUtilsTests.cs
+++ b/Packages/com.whisper.unity/Tests/Runtime/AudioUtilsTests.cs
@@ -30,8 +30,8 @@ public void ConvertToMonoTest()
         [Test]
         public void ChangeSampleRateTest()
         {
-            // create mono audio of 5 sec length
-            const int time = 5;
+            // create mono audio of 10 sec length
+            const int time = 10 * 60;
             const int srcRate = 16000;
             var src = new float[srcRate * time];