Selecting target by GUID

[NitroGlycerine]

I'm trying to target a unit by GUID using the following function:

00524BF0    CGGameUI__Target

I'm using the following code, based on some pieces of code I found on this forum:

        public static void SelectTarget( ulong guid )
        {
            Wow.SuspendThread(Wow.ThreadHandle);

            uint codecave = Wow.AllocateMemory(),
                dwHiWord = (uint)(guid >> 32),
                dwLoWord = (uint)guid;

            Wow.Asm.Clear();
            Wow.Asm.AddLine("MOV EAX, 0x{0}", dwHiWord.ToString("X"));
            Wow.Asm.AddLine("PUSH EAX");
            Wow.Asm.AddLine("MOV EAX, 0x{0}", dwLoWord.ToString("X"));
            Wow.Asm.AddLine("PUSH EAX");
            Wow.Asm.AddLine("CALL 0x00524BF0");
            Wow.Asm.AddLine("ADD ESP, 0x08");
            Wow.Asm.AddLine("RETN");

            Wow.Asm.InjectAndExecute(codecave);
            Wow.FreeMemory(codecave);
            Wow.ResumeThread();
        }

Wow throws a critical error when I call the SelectTarget function. I have verified that the GUID I pass to the function is a valid one. I'm still learning, and not really good on this topic yet. Could anyone point me to my mistake(s)? Thanks!

[MaiN]

You are not doing this in the main thread. That's your mistake.

[NitroGlycerine]

I'm using BlackMagic:

public bool OpenProcessAndThread(int dwProcessId)
    Member of Magic.BlackMagic

Summary:
Opens a process and its main thread for manipulation.

Parameters:
dwProcessId: Id of the target process.

I assumed that I was already going from the main thread.

[zys924]

Oh... It's "running" in the main thread, not just grab the handle of it...
You have to hook the EndScene and poke your code there to be executed by DX3D

[RedArray]

You should just create a delegate for the SelectTarget function and call it like a normal function instead of dealing with assembly imho.

[miceiken]

You should just create a delegate for the SelectTarget function and call it like a normal function instead of dealing with assembly imho.

I don't think he's inprocess

[RedArray]

I don't think he's inprocess

Oh thought BlackMagic was the in-process version. I dunno maybe not. If I recall there is two versions of basically the same thing. Ones in-process the other out.

[miceiken]

Oh thought BlackMagic was the in-process version. I dunno maybe not. If I recall there is two versions of basically the same thing. Ones in-process the other out.

Nah, BlackMagic is an oop memory reading library

[Cypher]

Oh thought BlackMagic was the in-process version. I dunno maybe not. If I recall there is two versions of basically the same thing. Ones in-process the other out.

You're thinking of WhiteMagic by Apoc.

[RedArray]

You're thinking of WhiteMagic by Apoc.

Ah yeah that's what it was. My bad.