RETAIL How to use Spell_C_HandleTerrainClick

[ostapus]

I am working on classic, and I too have this same problem. Do you get a divide by zero error? I'm still learning about native reversing, but I'm under the assumption the crash might be due to checking the call stack.
So it throws an error when not called from the game's actual address space. Thus why I can call the wrapper functions without an issue, but trying to directly call the function from my injected dll will flag the call stack check and crash.

I do use wrapper as well, and have no issues to call other function which does check return address. but calling the same way this (ClickToMove) function cause the crash. So i wonder if there are other checks unless i missing something
Hard to track the issues especcifaly when code is obfuscated.
Speaking about obfuscation - is there way to somehow automatically deobfuscate the code ?

tia

[GlittPrizes]

Can anyone provide OnTerrainClick offset or pattern for (38134)? I have a pattern for it, but I think it might be incorrect because the function is __int64 __fastcall OnTerrainClick(__int64 a1). And is it called with mover pointer followed by the location? It's hard to follow this thread whether which function you are talking about?

edit: okay so I was using the correct offset after all. I pass the mover correctly, my coords are correct, but my click is sent into out space instead of clicking nearby... Any ideas?

[ostapus]

Can anyone provide OnTerrainClick offset or pattern for (38134)? I have a pattern for it, but I think it might be incorrect because the function is __int64 __fastcall OnTerrainClick(__int64 a1). And is it called with mover pointer followed by the location? It's hard to follow this thread whether which function you are talking about?

edit: okay so I was using the correct offset after all. I pass the mover correctly, my coords are correct, but my click is sent into out space instead of clicking nearby... Any ideas?

i've 0x0000000141AD28E0 for OnTerrainClick... works for me. you have to check "click to move" options in interface options.

[GlittPrizes]

Thanks I was able to get it working after passing zeros to the mover part.

[imnothonorbuddy]

if call it in same point twice, it will crash game lol

[imnothonorbuddy]

OnTerrainClick randomly crashes, by the way, I added a vector length check between the last and next walking points (6 meters) and added a 500ms time between clicks. BUT it's still randomly crashes) but this happens less often. im internally

[Mathex]

OnTerrainClick randomly crashes, by the way, I added a vector length check between the last and next walking points (6 meters) and added a 500ms time between clicks. BUT it's still randomly crashes) but this happens less often. im internally

It doesn't for me. What does your click data struct look like? I basically just use

struct ClickData
{
    uintptr_t guid_high;
    uintptr_t guid_low;
    Vector3 location;
    int32_t click_type;
};

And always parse 0 for the guid fields and the click type varies of course.

[imnothonorbuddy]

It doesn't for me. What does your click data struct look like? I basically just use

struct ClickData
{
    uintptr_t guid_high;
    uintptr_t guid_low;
    Vector3 location;
    int32_t click_type;
};

And always parse 0 for the guid fields and the click type varies of course.

i have this struct

struct click_data
{
	uint64_t mover_1;
	uint64_t mover_2;
	vec3 location;
	int click_type;
};

and i click on terrain with

click_data point;
point.mover_1 = 0;
point.mover_2 = 0;
point.location = next_pos;
point.click_type = 0x4;