summaryrefslogtreecommitdiffstats
path: root/src/ClientHandle.cpp
diff options
context:
space:
mode:
authorMattes D <github@xoft.cz>2014-05-26 10:14:05 +0200
committerMattes D <github@xoft.cz>2014-05-26 10:14:05 +0200
commit6ad411a8b8878cfa66d4f88419e9dace80fdab2a (patch)
treeb15836509fd1f6c58ade91dea70aa47731ea367b /src/ClientHandle.cpp
parentSEMICOLONS! (diff)
parentAdd more move checks (diff)
downloadcuberite-6ad411a8b8878cfa66d4f88419e9dace80fdab2a.tar
cuberite-6ad411a8b8878cfa66d4f88419e9dace80fdab2a.tar.gz
cuberite-6ad411a8b8878cfa66d4f88419e9dace80fdab2a.tar.bz2
cuberite-6ad411a8b8878cfa66d4f88419e9dace80fdab2a.tar.lz
cuberite-6ad411a8b8878cfa66d4f88419e9dace80fdab2a.tar.xz
cuberite-6ad411a8b8878cfa66d4f88419e9dace80fdab2a.tar.zst
cuberite-6ad411a8b8878cfa66d4f88419e9dace80fdab2a.zip
Diffstat (limited to 'src/ClientHandle.cpp')
-rw-r--r--src/ClientHandle.cpp74
1 files changed, 58 insertions, 16 deletions
diff --git a/src/ClientHandle.cpp b/src/ClientHandle.cpp
index 83b21ae3c..e7537d74d 100644
--- a/src/ClientHandle.cpp
+++ b/src/ClientHandle.cpp
@@ -582,16 +582,37 @@ void cClientHandle::HandlePlayerPos(double a_PosX, double a_PosY, double a_PosZ,
return;
}
- /*
- // TODO: Invalid stance check
- if ((a_PosY >= a_Stance) || (a_Stance > a_PosY + 1.65))
+ if (m_Player->GetHealth() <= 0)
{
- LOGD("Invalid stance");
- SendPlayerMoveLook();
+ // The player is dead. He can't move :D
return;
}
- */
-
+
+ if (std::isnan(a_PosX) || std::isnan(a_PosY) || std::isnan(a_PosZ) || std::isnan(a_Stance))
+ {
+ LOGWARN("%s was caught trying to crash the server with an invalid position.", m_Player->GetName().c_str());
+ Kick("Nope!");
+ return;
+ }
+
+ // Invalid stance check
+ if (!m_Player->IsInBed())
+ {
+ double Difference = a_Stance - a_PosY;
+ if ((Difference > 1.65) || (Difference < 1.0))
+ {
+ Kick("Illegal stance!");
+ LOGWARN("%s had an illegal stance: %f", m_Player->GetName().c_str(), a_Stance);
+ return;
+ }
+ }
+
+ if ((std::abs(a_PosX) > 32000000.0) || (std::abs(a_PosZ) > 32000000.0))
+ {
+ Kick("Illegal position!");
+ return;
+ }
+
// If the player has moved too far, "repair" them:
Vector3d Pos(a_PosX, a_PosY, a_PosZ);
if ((m_Player->GetPosition() - Pos).SqrLength() > 100 * 100)
@@ -1389,7 +1410,7 @@ void cClientHandle::HandlePlayerLook(float a_Rotation, float a_Pitch, bool a_IsO
-void cClientHandle::HandlePlayerMoveLook(double a_PosX, double a_PosY, double a_PosZ, double a_Stance, float a_Rotation, float a_Pitch, bool a_IsOnGround)
+void cClientHandle::HandlePlayerMoveLook(double a_PosX, double a_PosY, double a_PosZ, double a_Stance, float a_Yaw, float a_Pitch, bool a_IsOnGround)
{
if ((m_Player == NULL) || (m_State != csPlaying))
{
@@ -1397,21 +1418,42 @@ void cClientHandle::HandlePlayerMoveLook(double a_PosX, double a_PosY, double a_
return;
}
- /*
- // TODO: Invalid stance check
- if ((a_PosY >= a_Stance) || (a_Stance > a_PosY + 1.65))
+ if (m_Player->GetHealth() <= 0)
{
- LOGD("Invalid stance");
- SendPlayerMoveLook();
+ // The player is dead. He can't move :D
+ return;
+ }
+
+ if (std::isnan(a_PosX) || std::isnan(a_PosY) || std::isnan(a_PosZ) || std::isnan(a_Stance))
+ {
+ LOGWARN("%s was caught trying to crash the server with an invalid position.", m_Player->GetName().c_str());
+ Kick("Nope!");
+ return;
+ }
+
+ // Invalid stance check
+ if (!m_Player->IsInBed())
+ {
+ double Difference = a_Stance - a_PosY;
+ if ((Difference > 1.65) || (Difference < 1.0))
+ {
+ Kick("Illegal stance!");
+ LOGWARN("%s had an illegal stance: %f", m_Player->GetName().c_str(), a_Stance);
+ return;
+ }
+ }
+
+ if ((std::abs(a_PosX) > 32000000.0) || (std::abs(a_PosZ) > 32000000.0))
+ {
+ Kick("Illegal position!");
return;
}
- */
m_Player->MoveTo(Vector3d(a_PosX, a_PosY, a_PosZ));
m_Player->SetStance (a_Stance);
m_Player->SetTouchGround(a_IsOnGround);
- m_Player->SetHeadYaw (a_Rotation);
- m_Player->SetYaw (a_Rotation);
+ m_Player->SetHeadYaw (a_Yaw);
+ m_Player->SetYaw (a_Yaw);
m_Player->SetPitch (a_Pitch);
}