more queryExtension return value checks, update cmake-conan

Server/Components/Pawn/Scripting/TextLabel/Natives.cpp

@@ -152,7 +152,12 @@ SCRIPT_API_FAILRET(CreatePlayer3DTextLabel, INVALID_TEXT_LABEL_ID, int(IPlayer&
 
 SCRIPT_API(DeletePlayer3DTextLabel, bool(IPlayer& player, IPlayerTextLabel& textlabel))
 {
-	queryExtension<IPlayerTextLabelData>(player)->release(textlabel.getID());
+	auto data = queryExtension<IPlayerTextLabelData>(player);
+	if (!data)
+	{
+		return false;
+	}
+	data->release(textlabel.getID());
 	return true;
 }
 

Server/Components/Pickups/pickups_main.cpp

@@ -120,7 +120,13 @@ class PickupsComponent final : public IPickupsComponent, public PlayerConnectEve
 			{
 				return false;
 			}
+
 			auto data = queryExtension<IPlayerPickupData>(peer);
+			if (!data)
+			{
+				return false;
+			}
+
 			int id = data->fromClientID(onPlayerPickUpPickupRPC.PickupID);
 			if (!id)
 			{

Server/Components/Vehicles/vehicle.cpp

@@ -322,6 +322,11 @@ bool Vehicle::updateFromTrailerSync(const VehicleTrailerSyncPacket& trailerSync,
 	updateOccupied();
 
 	PlayerVehicleData* playerData = queryExtension<PlayerVehicleData>(player);
+	if (!playerData)
+	{
+		return false;
+	}
+
 	Vehicle* vehicle = static_cast<Vehicle*>(playerData->getVehicle());
 
 	if (!vehicle || vehicle->detaching)
@@ -369,6 +374,10 @@ bool Vehicle::updateFromTrailerSync(const VehicleTrailerSyncPacket& trailerSync,
 bool Vehicle::updateFromPassengerSync(const VehiclePassengerSyncPacket& passengerSync, IPlayer& player)
 {
 	PlayerVehicleData* data = queryExtension<PlayerVehicleData>(player);
+	if (!data)
+	{
+		return false;
+	}
 	// Only do heavy processing if switching vehicle or switching between driver and passenger
 	int passengerSeats = Impl::getVehiclePassengerSeats(getModel());
 	// TODO: Deal with two players in the same seat.

Server/Components/Vehicles/vehicles_impl.hpp

@@ -91,7 +91,7 @@ class VehiclesComponent final : public IVehiclesComponent, public CoreEventHandl
 			}
 
 			IPlayerVehicleData* vehData = queryExtension<IPlayerVehicleData>(peer);
-			if (!lock.entry->isStreamedInForPlayer(peer) || !(peer.getState() == PlayerState_Driver || peer.getState() == PlayerState_Passenger) || vehData->getVehicle() != lock.entry)
+			if (vehData == nullptr || !lock.entry->isStreamedInForPlayer(peer) || !(peer.getState() == PlayerState_Driver || peer.getState() == PlayerState_Passenger) || vehData->getVehicle() != lock.entry)
 			{
 				return false;
 			}
@@ -126,6 +126,11 @@ class VehiclesComponent final : public IVehiclesComponent, public CoreEventHandl
 			}
 
 			PlayerVehicleData* data = queryExtension<PlayerVehicleData>(peer);
+			if (!data)
+			{
+				return false;
+			}
+
 			IVehicle* vehicle = data->getVehicle();
 			if (vehicle && vehicle->getDriver() == &peer)
 			{

Server/Source/player_pool.hpp

@@ -1494,7 +1494,7 @@ struct PlayerPool final : public IPlayerPool, public NetworkEventHandler, public
 			Player& player = static_cast<Player&>(peer);
 			PlayerState state = player.getState();
 			IPlayerVehicleData* vehData = queryExtension<IPlayerVehicleData>(peer);
-			if (state != PlayerState_Driver || vehData->getVehicle() == nullptr)
+			if (state != PlayerState_Driver || vehData == nullptr || vehData->getVehicle() == nullptr)
 			{
 				return false;
 			}