Fixed index out of range by adding nullcheck to read/write params. Null cached params when ownership to prevent another out-of-range.

Author: angusmf

MLAPI/MonoBehaviours/Prototyping/NetworkedAnimator.cs

@@ -22,7 +22,6 @@ public Animator animator
             set
             {
                 m_Animator = value;
-                m_AnimatorParameters = m_Animator.parameters;
                 ResetParameterOptions();
             }
         }
@@ -75,6 +74,7 @@ public void ResetParameterOptions()
         {
             Debug.Log("ResetParameterOptions");
             m_ParameterSendBits = 0;
+            m_AnimatorParameters = null;
         }
 
         void FixedUpdate()
@@ -248,6 +248,7 @@ internal void HandleAnimTriggerMsg(int clientId, byte[] data)
 
         void WriteParameters(BinaryWriter writer, bool autoSend)
         {
+            if (m_AnimatorParameters == null) m_AnimatorParameters = m_Animator.parameters;
             for (int i = 0; i < m_AnimatorParameters.Length; i++)
             {
                 if (autoSend && !GetParameterAutoSend(i))
@@ -278,7 +279,8 @@ void WriteParameters(BinaryWriter writer, bool autoSend)
         }
 
         void ReadParameters(BinaryReader reader, bool autoSend)
-        {     
+        {
+            if (m_AnimatorParameters == null) m_AnimatorParameters = m_Animator.parameters;
             for (int i = 0; i < m_AnimatorParameters.Length; i++)
             {
                 if (autoSend && !GetParameterAutoSend(i))
@@ -337,5 +339,15 @@ public void SetTrigger(int hash)
                 }
             }
         }
+
+        public override void OnGainedOwnership()
+        {
+            ResetParameterOptions();
+        }
+
+        public override void OnLostOwnership()
+        {
+            ResetParameterOptions();
+        }
     }
 }