SA-MP Forums

Go Back   SA-MP Forums > SA-MP > Bug Reports

Thread Tools Display Modes
Old 11/01/2018, 08:27 PM   #1
IstuntmanI's Avatar
Join Date: Mar 2009
Location: Romania
Posts: 1,885
Reputation: 370
Default OnPlayerWeaponShot when shooting a driver/passenger

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.

Last edited by IstuntmanI; 11/01/2018 at 10:07 PM.
IstuntmanI is offline   Reply With Quote
Old 20/01/2018, 05:32 PM   #2
Little Clucker
Join Date: Jul 2013
Posts: 12
Reputation: -1
Lightbulb Re: OnPlayerWeaponShot when shooting a driver/passenger

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.
Paglia is offline   Reply With Quote
Old 22/01/2018, 01:17 AM   #3
Gammix's Avatar
Join Date: Jan 2015
Location: Ottawa, Canada
Posts: 1,568
Reputation: 704
Default Re: OnPlayerWeaponShot when shooting a driver/passenger

This actually might mess up libraries using OnPlayerWeaponShot, specifically anticheats.
Gammix is online now   Reply With Quote

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Similar Threads
Thread Thread Starter Forum Replies Last Post
SA:MP Passenger shooting from car (By H) fix? Riwerry Scripting Help 16 06/05/2015 05:18 PM
How to show this for the driver and the passenger ? Drago987 Scripting Help 9 22/06/2014 03:44 AM
OnPlayerWeaponShot on DriveBy passenger Kyra Bug Reports 7 24/04/2014 05:39 PM
Help, Shooting from a car as passenger. Kenzix Scripting Help 5 18/03/2014 08:42 PM
Car take damage without a passenger/driver. Unri Scripting Help 4 26/09/2013 11:39 PM

All times are GMT. The time now is 06:29 AM.

Powered by vBulletin® Version 3.8.6
Copyright ©2000 - 2018, Jelsoft Enterprises Ltd.