-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Redux 0.9.1 cause game crash when call Memory function with string param #36
Comments
It's cause crash with function _send_chat_message. |
Can you enable LogOpcodes option in cleo.ini and show me the cleo_redux.log? |
|
Another problem with Memory.CallMethod is when i call
...funcParams must be [0, _color, 0, _message, 8] but it was [8, _message, 0, _color, 0] |
where do you see
Int(0), Int(16777215), Int(0), String("/time"), Int(8) |
Also passing string arguments into call functions is not supported, I'm wondering how it could have worked in 0.9.0, since those methods only accept integer numbers as arguments |
I converted a snippets from ugbase.eu
I have reversed the params to correct it. screen-recorder-thu-feb-24-2022-23-35-32.mp4 |
In JavaScript you should pass the arguments in the same order your target function expects. In SCM code they are reversed, but not in JS. Does it make sense? |
So if Don't pay much attention to the order in the log |
I think the only issue here is that you pass string argument directly to the function. It should not work. Was it working 100% in 0.9.0? |
It was. 100%. screen-recorder-thu-feb-24-2022-23-43-38.mp4 |
Ok, I will look into it |
thanks for the detailed report |
in 0.9.0 call_method was handled by CLEO Library that supports string arguments for this command. That is why your script worked. Since 0.9.1 CLEO Redux handles call_method command with its own implementation that only supports number arguments. That is what confused me, as I knew string arguments have not been supported yet. I will look into adding string arguments support in CLEO Redux. |
@tntd2k2 please test with 0.9.2 |
@x87 Now Memory.Read() return wrong value with dll. |
@tntd2k2 thanks for the report. I need to clarify a few things first.
|
|
Sometimes it crash when i use Memory function with game address.
Redux_log:
|
It's difficult to narrow down a particular problem especially dealing with screenshots. Let's focus on a single command that is not working in your opinion. I assume it's let _chat_info = Memory.Read(_library + _SAMP_CHAT_INFO_OFFSET_03DL, 4, true);
Please test with 0.9.2 |
const _library = DynamicLibrary.Load('samp.dll')
const _SAMP_CHAT_INFO_OFFSET_03DL = 0x2ACA10 2, 3. I think it was problem in DynamicLibrary.Load DynamicLibrary.Load('samp.dll') = 497865208 Memory.Read(_library, 4, true) = 61734912 (give another value when reload script)
samp.dll value in cheat engine = 9460301 (4 bytes) |
Are you using dylib plugin ? Check if there is dylib.cleo in CLEO_PLUGINS. If yes, try without it. |
@x87 It's work well when remove dylib.cleo (include string argument) |
so |
That's right. |
This is when i run with redux 0.9.0
ans with redux 0.9.1
This is my code
The text was updated successfully, but these errors were encountered: