[The__]

Okay, first off.

pawn Код:

if(IsPlayerInVehicle(playerid,carid) && CarInfo[carid][cOwned] == 1)

You have this, but right above it you already check to see if a player is in a vehicle.

Secondly,
OnPlayerUpdate(playerid) is called every second, meaning you don't even need this there.

Thirdly, what is carid defined as? I believe your command should be checking for ownership of the vehicle

pawn Код:

if(IsPlayerInVehicle(playerid,carid) && CarInfo[carid][cOwned] == 1)

should be checking if the vehicle is owned by the player or not, which the line above isn't right to check that.

Fourthly, I recommend ZCMD as it's faster/quicker to code, and less lines.