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

Breaks NetStats_ConnectionStatus #19

Closed
JernejL opened this issue Oct 24, 2015 · 4 comments
Closed

Breaks NetStats_ConnectionStatus #19

JernejL opened this issue Oct 24, 2015 · 4 comments

Comments

@JernejL
Copy link

JernejL commented Oct 24, 2015

NetStats_ConnectionStatus once ysf is loaded starts to return what looks like garbage.

[debug] Run time error 4: "Array index out of bounds"
[debug] Accessing element at negative index -1454227448

-1454227448 is result from NetStats_ConnectionStatus which should be only one of 0 - 10 positive values found here: http://wiki.sa-mp.com/wiki/Connectionstatus

Which hook is responsible for this?

@ghost
Copy link

ghost commented Oct 25, 2015

Going to check it soon.. Currently I'm away from computer. Maybe try to remove this line: https://github.com/kurta999/YSF/blob/YSF_/src/Hooks.cpp#L873

@JernejL
Copy link
Author

JernejL commented Oct 26, 2015

Removed that, i'm not sure if it fixed it since i've commented out half the plugin because removing "a few hooks" kinda excelated. My timers stopped working after a while when i started to use the plugin and that triggered a chain reaction (i removed a few more hooks that i suspected could be related and then suddendly gang zones stopped working, so i kept trimming it down to minimum on what i needed to keep just the getpacketid hook - i use the hook to override keys there and not have the plugin crash the server).

Ofcourse a better way for me to do it would be in CHookRakServer::Send -> RebuildSyncData, but there is no code in there yet to support rebuilding ID_VEHICLE_SYNC..

@ghost
Copy link

ghost commented Oct 30, 2015

Going to fix this + going to remove logprintf hook from linux, because that is unstable when lot of calls happen. IDK why.

@JernejL
Copy link
Author

JernejL commented Oct 31, 2015

Onplayerconnect -> amx error -> crashdetect wants to output a message, ysf hooked it and called OnServerMessage, but pawn was already in a public call, crashdetect takes the problem seriously and calls exception again, printf hook proceeds to repeat.. until crash.

@ghost ghost closed this as completed Jan 25, 2016
@djacicd djacicd mentioned this issue Nov 21, 2017
@dnl-k dnl-k mentioned this issue Apr 27, 2018
This issue was closed.
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

1 participant