Fraktionsfahrzeuge anzeigen lassen

Weil du über dein Array hinausläufst.

Ausserdem formatierst du str immer wieder um.

new var;
for(new i=0;i<max_vehicles;i++)
{
if(FrakCarInfo[i][f_Frak] == GetPVarInt(playerid,"Fraktion")) // <- Dein Array von FrakCarInfo ist 100%ig kleiner als der Wert max_vehicles.
{
format(str,144,"- %s\n",GetVehicleName(FrakCarInfo[i][f_car])); // <- Pro schleife überschreibst du das ding immer neu, mal abgesehen davon, dass du auch den Client mit Dialogen zubombst.
ShowPlayerDialog(playerid,DIALOG_FCAR,DIALOG_STYLE_LIST,"Fraktionsfahrzeuge",str,"OK","");
var ++;
return 1;
}
}
if(!var) SCM(playerid,COLOR_RED,"Zurzeit besitzt deine Fraktion keine Fahrzeuge!");
return 1;

du darfst in der for-schleife kein return 1;
benutzen, dabei wird die Funktion beendet und die for-schleife läuft nur 1-mal durch

ne menge sogar:
COMMAND:fcars(playerid,params[])
{
new var;

for(new i=0;i<MAX_VEHICLES;i++) // <- Das musst du auf die größe von FrakCarInfo anpassen
{
if(FrakCarInfo[i][f_Frak] == GetPVarInt(playerid,"Fraktion"))
{
format(str,144,"- %s\n",GetVehicleName(FrakCarInfo[i][f_car])); // Das musst du umschreiben, da es immer neu formatiert wird
ShowPlayerDialog(playerid,DIALOG_FCAR,DIALOG_STYLE_LIST,"Fraktionsfahrzeuge",str,"OK",""); // Das solltest du auch umschreiben, sonst bekommste pro durchgang 1 Dialog
var ++;
return 1; // <- Das muss weg
}
}
if(!var) SCM(playerid,COLOR_RED,"Zurzeit besitzt deine Fraktion keine Fahrzeuge!");
return 1;
}