From 16aeb84cd35996a6b41f10cbc48a677eeccc911c Mon Sep 17 00:00:00 2001 From: Tiger Wang Date: Sat, 2 Jan 2021 13:50:34 +0000 Subject: Fix potential destruction crashes (#5095) * Fix potential destruction crashes * Fix destructors accessing destroyted objects * Fix cPlayer not destroying windows (Destroyed never called) * Tentatively fixes #4608, fixes #3236, fixes #3262 - Remove cEntity::Destroyed() and replace with cEntity::OnRemoveFromWorld() * Add missing call to OnRemoveFromWorld --- src/Mobs/Monster.cpp | 44 ++++++++++---------------------------------- 1 file changed, 10 insertions(+), 34 deletions(-) (limited to 'src/Mobs/Monster.cpp') diff --git a/src/Mobs/Monster.cpp b/src/Mobs/Monster.cpp index 048393e67..7e8e7eba3 100644 --- a/src/Mobs/Monster.cpp +++ b/src/Mobs/Monster.cpp @@ -135,17 +135,15 @@ cMonster::cMonster(const AString & a_ConfigName, eMonsterType a_MobType, const A -cMonster::~cMonster() +void cMonster::OnRemoveFromWorld(cWorld & a_World) { - ASSERT(GetTarget() == nullptr); -} - - - + SetTarget(nullptr); // Tell them we're no longer targeting them. + if (m_LovePartner != nullptr) + { + m_LovePartner->ResetLoveMode(); + } -void cMonster::OnRemoveFromWorld(cWorld & a_World) -{ if (IsLeashed()) { cEntity * LeashedTo = GetLeashedTo(); @@ -165,20 +163,6 @@ void cMonster::OnRemoveFromWorld(cWorld & a_World) -void cMonster::Destroyed() -{ - SetTarget(nullptr); // Tell them we're no longer targeting them. - if (m_LovePartner != nullptr) - { - m_LovePartner->ResetLoveMode(); - } - Super::Destroyed(); -} - - - - - void cMonster::SpawnOn(cClientHandle & a_Client) { a_Client.SendSpawnMob(*this); @@ -762,22 +746,14 @@ void cMonster::CheckEventSeePlayer(cChunk & a_Chunk) const auto MyHeadPosition = GetPosition().addedY(GetHeight()); // Enumerate all players within sight: - m_World->ForEachEntityInBox({ GetPosition(), m_SightDistance * 2.0 }, [this, &TargetPlayer, &ClosestDistance, MyHeadPosition](cEntity & a_Entity) + m_World->ForEachPlayer([this, &TargetPlayer, &ClosestDistance, MyHeadPosition](cPlayer & a_Player) { - if (!a_Entity.IsPlayer()) - { - // Continue iteration: - return false; - } - - const auto Player = static_cast(&a_Entity); - - if (!Player->CanMobsTarget()) + if (!a_Player.CanMobsTarget()) { return false; } - const auto TargetHeadPosition = a_Entity.GetPosition().addedY(a_Entity.GetHeight()); + const auto TargetHeadPosition = a_Player.GetPosition().addedY(a_Player.GetHeight()); const auto TargetDistance = (TargetHeadPosition - MyHeadPosition).SqrLength(); // TODO: Currently all mobs see through lava, but only Nether-native mobs should be able to. @@ -786,7 +762,7 @@ void cMonster::CheckEventSeePlayer(cChunk & a_Chunk) cLineBlockTracer::LineOfSightTrace(*GetWorld(), MyHeadPosition, TargetHeadPosition, cLineBlockTracer::losAirWaterLava) ) { - TargetPlayer = Player; + TargetPlayer = &a_Player; ClosestDistance = TargetDistance; } -- cgit v1.2.3