Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Profiler Crashes After Function Call #21

Open
JonRB opened this issue Dec 8, 2016 · 1 comment
Open

Profiler Crashes After Function Call #21

JonRB opened this issue Dec 8, 2016 · 1 comment

Comments

@JonRB
Copy link

JonRB commented Dec 8, 2016

When I call the GivePlayerWeaponEx function in my code, the plugin sometimes crashes.

Additionally, when the gamemode exits in any way, the server crashes and no output is produced.

I tried local builds (Debug and Release) but they bore no fruit.

The only version I have managed to get results out of is 2.10.3, which seems to be giving rather anomalous results for the above mentioned function - Apparently it is using up 64.61% of the Total Time, and given that the GivePlayerWeaponEx function does some very straightforward ammo checking (To avoid the GivePlayerWeapon ammo level issue in which you can give somebody one ammo of a different weapon and they will have 1+N, where N is their old ammo level).

I'd love to get this plugin running so that I can better diagnose server issues.

@Zeex
Copy link
Owner

Zeex commented Dec 11, 2016

Can you copy the stack trace from your local debug build and post it here so I could see where it crashed? You'll need to run samp-server.exe under Visual Studio to do this.

Zeex added a commit that referenced this issue Apr 7, 2018
fda6940 Set up asm compilation for occasional manual testing
ded81e7 Add support for NOP (one-byte version) (#28)
698d230 Remove broken version badge
d2ac83c Replace 0 with NULL
0627975 Fix CMake warning about @rpath
3fa84a3 Merge pull request #27 from Y-Less/master
e120562 Made all subhook calls NULL safe, not just one destructor.
8046f66 Merge pull request #26 from tharindud/master
f4614ac Fixing a compiler error in the example C++ code in README.md due to '<<' being specified as '<'
699a6bd Merge pull request #23 from gocha/fix-dtor-nullref
2f9d0c1 Merge pull request #24 from gocha/scoped-install-with-new-handler
46ebdb5 Add new ScopedHookInstall constructor which can set/override the new hook destination
1edbc72 Fix null reference on the destruction of empty Hook
50a8d0c Merge pull request #21 from omgtehlion/patch-1
5607ef9 SUB r/m32, imm32 instruction typo
31a4f0f v0.4
ee6b547 Revert "Enable tests on AppVeyor"
95cd7b4 Properly decode SIB length
0353549 Cleanup
52af4ed Add more logging in test program
e3f5597 Tell CTest to show output on failure
af522f7 Add support for Mac OS
1503a23 Add x64 platform for AppVeryor
2428b50 Configure 32-bit Travis build
1ec5591 Enable tests on AppVeyor
cd97522 Fix relocation bug
c86c12a Add support for REX prefixes
889413d Fix CMake errors
c72e0a2 Return error on overflow
e7a7b14 Remove stupid assert
4f36fb1 Fix test program
d4f4d17 Fix trampoline size calculation
a8d7b78 Enable 64-bit offsets in test program
cdb4c0e Make sure offset fits in 32 bits
93d167a Fix trampoline generation
a55f789 Fix 64-bit jump being overwritten by regular jump
866f38d Update example code
bfdb2b4 Improve C++ interface
cba3854 Fix compile errors in C++ wrapper
d2a5756 Minor cleanup
384cb35 v0.3
bda8b5f Convert tabs to spaces
1b89e7f Return support for 32-bit jumps on x86-64
9551217 Fix pointer cast to smaller type warning
4d2ab7b Cleanup
25ee731 Fix MSVC compile errors
7484593 Merge pull request #13 from RomanHargrave/master
4e1c767 Modify x64 support to use RET rather than JMP

git-subtree-dir: deps/subhook
git-subtree-split: fda6940c20d862bf63c3bfffd16a5a9380b5a471
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants