[IstuntmanI]

If you are on foot and you shoot a driver/passenger in a vehicle (and damaging him) OnPlayerWeaponShot will only be called with hittype = BULLET_HIT_TYPE_VEHICLE with the hitid = player's vehicleid. The vehicle isn't actually shot at all and it isn't receiving damage and the driver/passenger is actually getting damage. In the "worst case" OnPlayerWeaponShot should be called twice: once for the vehicle and once for the damaged player.

I tried to find if this was already reported, but only the one about driveby-ing without calling the callback was reported.

[Paglia]

I don't know if this will be of much help but it may be somewhat related to this issue.

When a player is on foot, then CPlayer* points to the player itself. However, when a player is on a vehicle, CPlayer* points to the vehicle the player is driving.

I suspect that this goes on in samp too, and that's why the hittype is BULLET_HIT_TYPE_VEHICLE and the hitid = player's vehicleid.

(Guess as to what happens below)
The game checks for the CPlayer * directly to see what the player id's is (keep in mind that in samp the player pointer points to the first element, the id) to see what the player id is, and since that player is in a vehicle, then the hitid becomes that vehicle's id, and therefore the type is assigned BULLET_HIT_TYPE_VEHICLE.

(A possible fix?)
In that part of the code where the callback is evaluated, there should be some check to see if the dereferenced CPlayer* points to the actual player in question. Perhaps this can be done by checking one of the "unique" members for each "object" and to determine that way if the dereferenced object in question is either a car or a player.

I apologize for my confusing input, but this hopefully gives idea on how to go in the right direction to solve this.

[Gammix]

This actually might mess up libraries using OnPlayerWeaponShot, specifically anticheats.

[RogueDrifter]

Quote:

Originally Posted by Gammix This actually might mess up libraries using OnPlayerWeaponShot, specifically anticheats.

You can check for hittype 2 and loop through all players if GetPlayerVehicleID(i) = hitid (hitid = vehicle id that was hit) (i = the player picked out the loop) then set a variable to mark that the player riding this vehicle got his vehicle shot and never use it in any other way because what he stated above will fuck it, so you need to detect shots shot for players driving under hittype 2 and a loop, then detect shots shot only for onfoot players (separating them via using GetPlayerState) not sure if u understand me but i used that on my in-process anti gm i'll release soon and i didnt get false positive for the matter that was stated above.

[Gammix]

Quote:

Originally Posted by RogueDrifter You can check for hittype 2 and loop through all players if GetPlayerVehicleID(i) = hitid (hitid = vehicle id that was hit) (i = the player picked out the loop) then set a variable to mark that the player riding this vehicle got his vehicle shot and never use it in any other way because what he stated above will fuck it, so you need to detect shots shot for players driving under hittype 2 and a loop, then detect shots shot only for onfoot players (separating them via using GetPlayerState) not sure if u understand me but i used that on my in-process anti gm i'll release soon and i didnt get false positive for the matter that was stated above.

[RogueDrifter]

Quote:

Originally Posted by Gammix

lmao, what i meant was, his problem is when a player is shot in a vehicle only hittype 2 is called, so what i was saying is why's that a problem when you can loop on hittype 2 for the driver of hitid (vehicle id) right? like this:

PHP Code:

public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ)
{
    switch(hittype)
    {
        case 1:
        {
            if(GetPlayerState(playerid) == 1)
            {
                //player hit on foot.
                }
            }
        case 2:
        {
            for(new i, j = GetPlayerPoolSize(); i <= j; i++)
            {
                if(!IsPlayerConnected(i) || GetPlayerState(i) == 1) continue;
                if(GetPlayerVehicleID(i) == hitid && GetPlayerState(i) ==2)
                {
                    //driver of the vehicle that was hit by another player is 'i'
                    }
                if(GetPlayerVehicleID(i) == hitid && GetPlayerState(i) ==3)
                {
                    //passenger of the vehicle shot
                    }
                }
            }
        }
    return 1;
} 


[Gammix]

Quote:

Originally Posted by RogueDrifter lmao, what i meant was, his problem is when a player is shot in a vehicle only hittype 2 is called, so what i was saying is why's that a problem when you can loop on hittype 2 for the driver of hitid (vehicle id) right? like this: PHP Code: public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ) {     switch(hittype)     {         case 1:         {             if(GetPlayerState(playerid) == 1)             {                 //player hit on foot.                 }             }         case 2:         {             for(new i, j = GetPlayerPoolSize(); i <= j; i++)             {                 if(!IsPlayerConnected(i) || GetPlayerState(i) == 1) continue;                 if(GetPlayerVehicleID(i) == hitid && GetPlayerState(i) ==2)                 {                     //driver of the vehicle that was hit by another player is 'i'                     }                 if(GetPlayerVehicleID(i) == hitid && GetPlayerState(i) ==3)                 {                     //passenger of the vehicle shot                     }                 }             }         }     return 1; }

[RogueDrifter]

I've made a fix for this: LINK