OpenSim
/
OpenSimMirror
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Stop explicitly closing and nulling out Animator in order to prevent NREs in various places due to race conditions. 

Even where checks are being made they aren't enough since they all assume that the Animator they just checked is still there in the next line, which is not necessarily the case without locking.
The memory used is small and these should be GC'd anyway when the SP is released.  If this is not happening then the wider problem of old SPs being retained needs to be resolved.
0.7.3-extended
Justin Clark-Casey (justincc) 2012-07-19 22:59:28 +01:00
parent 47e2922a40
commit 1f8ac33ecb
2 changed files with 6 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
OpenSim/Region/Framework/Scenes/Animation/ScenePresenceAnimator.cs
View File

@ -524,11 +524,5 @@ namespace OpenSim.Region.Framework.Scenes.Animation
SendAnimPack(animIDs, sequenceNums, objectIDs);
}
public void Close()
{
m_animations = null;
m_scenePresence = null;
}
}
}

22
OpenSim/Region/Framework/Scenes/ScenePresence.cs
View File

@ -102,15 +102,10 @@ namespace OpenSim.Region.Framework.Scenes
public UUID currentParcelUUID = UUID.Zero;
protected ScenePresenceAnimator m_animator;
/// <value>
/// The animator for this avatar
/// </value>
public ScenePresenceAnimator Animator
{
get { return m_animator; }
private set { m_animator = value; }
}
public ScenePresenceAnimator Animator { get; private set; }
/// <summary>
/// Attachments recorded on this avatar.
@ -2631,8 +2626,7 @@ namespace OpenSim.Region.Framework.Scenes
//m_log.DebugFormat("[SCENE PRESENCE] SendAvatarDataToAgent from {0} ({1}) to {2} ({3})", Name, UUID, avatar.Name, avatar.UUID);
avatar.ControllingClient.SendAvatarDataImmediate(this);
if (Animator != null)
Animator.SendAnimPackToClient(avatar.ControllingClient);
Animator.SendAnimPackToClient(avatar.ControllingClient);
}
/// <summary>
@ -3300,14 +3294,12 @@ namespace OpenSim.Region.Framework.Scenes
//if ((Math.Abs(Velocity.X) > 0.1e-9f) || (Math.Abs(Velocity.Y) > 0.1e-9f))
// The Physics Scene will send updates every 500 ms grep: PhysicsActor.SubscribeEvents(
// as of this comment the interval is set in AddToPhysicalScene
if (Animator != null)
{
// if (m_updateCount > 0)
// {
Animator.UpdateMovementAnimations();
Animator.UpdateMovementAnimations();
// m_updateCount--;
// }
}
CollisionEventUpdate collisionData = (CollisionEventUpdate)e;
Dictionary<uint, ContactPoint> coldata = collisionData.m_objCollisionList;
@ -3322,7 +3314,7 @@ namespace OpenSim.Region.Framework.Scenes
// m_lastColCount = coldata.Count;
// }
if (coldata.Count != 0 && Animator != null)
if (coldata.Count != 0)
{
switch (Animator.CurrentMovementAnimation)
{
@ -3393,7 +3385,7 @@ namespace OpenSim.Region.Framework.Scenes
ControllingClient.SendHealth(Health);
}
public void Close()
protected internal void Close()
{
// Clear known regions
KnownRegions = new Dictionary<ulong, string>();
@ -3409,8 +3401,6 @@ namespace OpenSim.Region.Framework.Scenes
// m_reprioritizationTimer.Dispose();
RemoveFromPhysicalScene();
Animator.Close();
Animator = null;
}
public void AddAttachment(SceneObjectGroup gobj)