More validity checks for vehicles (mainly trailers) (#985)

* Check for non-streamed trailer

* Check driver and passenger sync

if the reported vehicle is streamed in

* Passenger sync without stream check

as train carriages always aren't streamed

* Add turnSpeed validation for trailers

Author: NexiusTailer

Server/Components/Vehicles/vehicle.cpp

@@ -159,7 +159,7 @@ void Vehicle::streamOutForClient(IPlayer& player)
 
 bool Vehicle::updateFromDriverSync(const VehicleDriverSyncPacket& vehicleSync, IPlayer& player)
 {
-	if (respawning)
+	if (respawning || !streamedFor_.valid(player.getID()))
 	{
 		return false;
 	}
@@ -233,7 +233,15 @@ bool Vehicle::updateFromDriverSync(const VehicleDriverSyncPacket& vehicleSync, I
 			trailer = static_cast<Vehicle*>(pool->get(vehicleSync.TrailerID));
 			if (trailer)
 			{
-				trailer->cab = this;
+				const bool isStreamedIn = trailer->isStreamedInForPlayer(player);
+				if (!isStreamedIn)
+				{
+					trailer = nullptr;
+				}
+				else
+				{
+					trailer->cab = this;
+				}
 			}
 		}
 	}

Server/Source/player_pool.hpp

@@ -1532,6 +1532,11 @@ struct PlayerPool final : public IPlayerPool, public NetworkEventHandler, public
 				return false;
 			}
 
+			if (trailerSync.TurnVelocity.x < -1.0f || trailerSync.TurnVelocity.x > 1.0f || trailerSync.TurnVelocity.y < -1.0f || trailerSync.TurnVelocity.y > 1.0f || trailerSync.TurnVelocity.z < -1.0f || trailerSync.TurnVelocity.z > 1.0f)
+			{
+				return false;
+			}
+
 			IVehicle* vehiclePtr = self.vehiclesComponent->get(trailerSync.VehicleID);
 			if (!vehiclePtr)
 			{