-
Notifications
You must be signed in to change notification settings - Fork 17
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
amx_fork/threaded crash [Linux] #4
Comments
Hmm, I am not able to reproduce the issue. Can you upload the executable so I can inspect it? |
If I'm right, you mean the compiled .so file? |
The crash is at a regular call to |
I also can't test the .so on my server, since it's missing some symbols. Can you build it with |
So yeah, the code doesn't crash if I remove the amx_forked block. |
I can't reproduce the issue even with the .so (which works though). Can you please send me the pwn/amx you are testing with? |
I would love too, but it is fairly large and requires a lot of includes to compile correctly (going for a "modular" approach). Could it be the command processor (Pawn.CMD) that is interfering with PawnPlus? |
I don't think the chance is too high, but perhaps you could isolate the issue into a portable filterscript. You could also test if there is a difference between these pieces of code: amx_forked(fork_exec)
{
print("inside");
} amx_forked(fork_data)
{
print("inside");
} amx_forked(fork_machine)
{
print("inside");
} |
Yeah, I just found out that it isn't caused by Pawn.CMD. Tried all of the pieces of code you listed and the only one that didn't crash (tested in main()) was fork_data, odd. |
Strange... Do you have other plugins that could affect the crash, like JIT for example? Does |
|
Okay, apparently |
crashdetect replaces the implementation of AMX with its own, so it basically controls every execution. I was probably using a version which doesn't cause an issue. Mine is CrashDetect v4.15.1. |
Okay, I have downloaded the latest version of CrashDetect and now I can reproduce the crash on Windows as well. Thanks for reporting! |
Anytime! I'll be downgrading to v4.15.1 for now, thanks for the help! |
If you can compile the latest commit of CrashDetect, can you check please if it works on Linux for you? |
Yup, all good now, compiled the latest commit of EDIT: Just wanted to apologize for making you wait this long, busy with something else. |
Hey again, not quite sure what the issue here is, but;
I've been looking through the documentation examples and have tried a few. A few work and a few just crash the plugin (server).
Tried using this in a command;
And I get a crash error;
The text was updated successfully, but these errors were encountered: