Fix #2906: warpPedIntoVehicle causes a C++ runtime assertion failure

Server/mods/deathmatch/logic/CPed.cpp

@@ -406,6 +406,9 @@ CVehicle* CPed::SetOccupiedVehicle(CVehicle* pVehicle, unsigned int uiSeat)
     static bool bAlreadyIn = false;
     if (!bAlreadyIn)
     {
+        if (uiSeat < MAX_VEHICLE_SEATS)
+            return m_pVehicle;
+
         // Store it
         m_pVehicle = pVehicle;
         m_uiVehicleSeat = uiSeat;

Server/mods/deathmatch/logic/CStaticFunctionDefinitions.cpp

@@ -4136,6 +4136,9 @@ bool CStaticFunctionDefinitions::WarpPedIntoVehicle(CPed* pPed, CVehicle* pVehic
     assert(pPed);
     assert(pVehicle);
 
+    if (pVehicle->GetMaxPassengers() == VEHICLE_PASSENGERS_UNDEFINED)
+        return false;
+
     // Valid seat id for that vehicle?
     // Temp fix: Disable driver seat for train carriages since the whole vehicle sync logic is based on the the player on the first seat being the vehicle
     // syncer (Todo)

Server/mods/deathmatch/logic/CVehicle.cpp

@@ -618,6 +618,9 @@ bool CVehicle::SetOccupant(CPed* pPed, unsigned int uiSeat)
     static bool bAlreadySetting = false;
     if (!bAlreadySetting)
     {
+        if (uiSeat < MAX_VEHICLE_SEATS)
+            return false;
+
         // Set the Player
         if (m_pOccupants[uiSeat] != pPed)
         {
@@ -644,7 +647,7 @@ bool CVehicle::SetOccupant(CPed* pPed, unsigned int uiSeat)
         return true;
     }
 
-    return true;
+    return true; // return true? even if function didn't end successfully?
 }
 
 void CVehicle::SetSyncer(CPlayer* pPlayer)