C++ Hilfe .. !

[Furkiii]

So Leute ich arbeite an einen DLL Hack für Nos doch da gibt es ein Problem sobald ich Nostale mit der DLL injectiere steht da Nostale.exe funktioniert nichtmehr blablabla ...

Hier der Code :

 Spoiler

PHP Code:

#include <Windows.h>
#include <iostream>

using namespace std;

DWORD BasePointer = 0; 
#define Points 0x000009C4

bool State = false; 
void punkte() 
{
        if(State)
    {
        DWORD dwBasePointer = *(DWORD*)BasePointer; 
        if(dwBasePointer != 0)
        {
            DWORD &dwPoints = *(DWORD*)(dwBasePointer +  Points);

            dwPoints = 4; 

        }

    }
    else 
    {
        DWORD dwBasePointer = *(DWORD*)BasePointer;
        if(dwBasePointer != 0) 
        {    
            DWORD &dwPoints = *(DWORD*)(dwBasePointer +  Points);

            dwPoints = 0; 

        }
    }
}
DWORD WINAPI HackThread(LPVOID unused)
{
    //---Hier wird unser BasePointer ausgerechnet !
    DWORD ImageBase = (DWORD)GetModuleHandle(0); 
    BasePointer = ImageBase + 0x004E6ABC;

    for(; ;)
    {
        punkte();
        if(GetAsyncKeyState(VK_F1)&1)State =! State;
        Sleep(10);
    }
}
BOOL WINAPI DllMain(HINSTANCE mod, DWORD DWORD_GRUND, LPVOID res)
{
    switch(DWORD_GRUND)
    {
    case DLL_PROCESS_ATTACH:
        CreateThread(0, 0, &HackThread , 0, 0, 0);  
         break;
     case DLL_PROCESS_DETACH:  
        MessageBoxA(0, "Not Loaded", "Info",0);
        break;
    }
    return TRUE;
} 


Es geht hier bei um einen Lifehack in den Minigames !

[Rorc]

#moved

[Elektrochemie]

Geb mal aus ob deine ImageBase Addresse bzw dein Basepointer richtig ist.
DWORD ImageBase = (DWORD)GetModuleHandle(0);
Das sieht mir bischen eigenartig aus, aber okay.


if(GetAsyncKeyState(VK_F1)&1)State =! State;

Warum machst du das per AND Vergleich?

[Nightblizard]

Da ich "Nos" nicht kenne zähle ich einfach mal alle potentiellen Fehlerquellen auf.

DWORD BasePointer = 0;

In x64 Prozessen kann das zu Fehlern führen, da dort Arbeitsspeicher mit 8 Byte adressiert wird. Zeigt dein Pointer nun z.B. auf C00000000, dann passt das nicht in die Variable und in ihr steht dann lediglich 0.

#define Points 0x000009C4

Mal ganz davon abgesehen, dass das absolut unschön ist; bist du dir sicher, dass die Adresse stimmt?

DWORD WINAPI HackThread(LPVOID unused)

Rückgabewert definiert, aber es wird nichts zurückgegeben. Dass das überhaupt erstellt werden kann wundert mich! Welchen Compiler nutzt du?

DWORD dwBasePointer = *(DWORD*)BasePointer;  
DWORD ImageBase = (DWORD)GetModuleHandle(0);

Siehe Punkt 1.

DWORD ImageBase = (DWORD)GetModuleHandle(0);

Bist du dir sicher, dass dies das richtige Modul ist?

BasePointer = ImageBase + 0x004E6ABC;

Bist du dir sicher, dass die Adresse stimmt?


Abgesehen davon solltest du dir einen einheitlichen Coding-Style angewöhnen, das schaut gruselig aus...



Letztenendes solltest du deinen Code ein wenig Debuggen und uns mehr Infos zukommen lassen, ansonsten können wir dir nicht wirklich helfen.

[XxharCs]

dein pointer offset (modul address) ist höchstwahrscheinlich falsch
also das: 0x004E6ABC

weil wenn die adresse falsch ist, erzeugt die dll injection nen overflow und deswegen stürzt nostale ab

Rückgabewert definiert, aber es wird nichts zurückgegeben. Dass das überhaupt erstellt werden kann wundert mich! Welchen Compiler nutzt du?

willst du mir sagen das du void HackThread(){... machst ? <- ist nähmlich das schlechteste was man tuhen kann

2.

DWORD WINAPI HackThread(LPVOID [B]unused[/B])

wie du siehst und man kann auch einen logischen schluss ziehen, verwenden wir den rückgabewert garnicht


noch zum TE:
den code copy&pasten und eigentlich nicht viel ahnung von dieser umgebung zu haben, passt nicht zusammen

[MrSm!th]

Originally Posted by XxharCs

dein pointer offset (modul address) ist höchstwahrscheinlich falsch
also das: 0x004E6ABC

weil wenn die adresse falsch ist, erzeugt die dll injection nen overflow und deswegen stürzt nostale ab

Das musst du mir nochmal genauer erklären.

willst du mir sagen das du void HackThread(){... machst ? <- ist nähmlich das schlechteste was man tuhen kann

Nein, er will sagen, dass man in einer Funktion, die als DWORD definiert ist, auch einen Wert vom Typ DWORD zurückgeben muss. Ein fehlendes return Statement ist ein Fehler.

2.

DWORD WINAPI HackThread(LPVOID [B]unused[/B])

wie du siehst und man kann auch einen logischen schluss ziehen, verwenden wir den rückgabewert garnicht

Scheiße, wenn man den Unterschied zwischen Parametern und Rückgabewerten nicht kennt.

[XxharCs]

Originally Posted by MrSm!th

Scheiße, wenn man den Unterschied zwischen Parametern und Rückgabewerten nicht kennt.

Klar kenn ich den

Man kann aber parameter auch zurück geben, aber nicht immer und nicht jeden ! (paar andere vorraussetzungen spielen da natürlich auch mit..)

ein bsp in java wär:

public static int method2(String s) throws IllegalArgumentException {
		if((s.charAt(0) > 'G') && (s.charAt(s.length() -1) < 'M')) {
			throw new IllegalArgumentException("So nicht.");
		}
		return s.length();
	}

Originally Posted by MrSm!th

Das musst du mir nochmal genauer erklären.

wenn man in ein spiel injectet und die adressen sind falsch, bekommt das spiel nen dc weil die dll auf irgendeine adresse zeigt etc.
ich hoffe das ich es besser erklärt hab, weil ich kein typ der alles 1a erklären kann

deswegen sag ich das seine adresse falsch ist

[Nightblizard]

Originally Posted by XxharCs

weil wenn die adresse falsch ist, erzeugt die dll injection nen overflow und deswegen stürzt nostale ab

Nein, das erzeugt keinen Overflow. Was da passiert nennt sich Zugriffsverletzung (access violation).

Overflows:
Pufferüberlauf
Arithmetischer Überlauf

Zugriffsverletzung:
Schutzverletzung

Originally Posted by XxharCs

willst du mir sagen das du void HackThread(){... machst ? <- ist nähmlich das schlechteste was man tuhen kann

Schön, dass du meine Aussage total missverstehst und dann eine solche Aussage in den Raum stellst ohne jede Begründung. ^^

Lass mich dir das mal kurz zeigen, denn so schlimm ist das eigentlich nicht.

long __stdacall foo(long unused)
{
}

void bar()
{
     auto i = foo(0);
}

Nun in Assembly:

;long __stcall foo(long unused)
push ebp
mov ebp, esp
pop ebp
retn 4

;void bar()
push ebp
mov ebp, esp
push ecx
push 0
call foo
mov [ebp-4], eax
mov esp, ebp
pop ebp
ret

So wie er es benutzt passiert da nichts schlimmes, es wird dann irgendein Wert in deine Variable gesteckt. Das Gleiche passiert, wenn du void als Rückgabetypen wählst. Im eax Register steht dann lediglich das, was vor dem Aufruf der Funktion drinnen stand.

Klar, das ist trotzdem falsch und es kann in x64 Prozessen crashen, aber ich habe das auch so gar nicht gemeint (wie Smith bereits sagte).

Originally Posted by XxharCs

2.

DWORD WINAPI HackThread(LPVOID [B]unused[/B])

wie du siehst und man kann auch einen logischen schluss ziehen, verwenden wir den rückgabewert garnicht

Hier haust du ein paar wirklich fundamentale Dinge durcheinander! Ich rate dir nochmal ein Buch zu nehmen und das Kapitel "Funktionen/Prozeduren" durchzuarbeiten.

Originally Posted by XxharCs

noch zum TE:
den code copy&pasten und eigentlich nicht viel ahnung von dieser umgebung zu haben, passt nicht zusammen

Nehm dir ein Beispiel dran. Wenn man keine Ahnung hat, dann sollte man auch keine Tipps geben.

Originally Posted by XxharCs

Man kann aber parameter auch zurück geben, aber nicht immer und nicht jeden ! (paar andere vorraussetzungen spielen da natürlich auch mit..)

Man gibt keine Parameter zurück, das ist absoluter Unsinn!

Originally Posted by XxharCs

ein bsp in java wär:

Wo genau gibst du da nun einen Parameter zurück? Du gibst in deinem Beispiel die Länge des Parameters "String s" zurück, sofern es den vorrausgegangenen Check übersteht.

Originally Posted by XxharCs

wenn man in ein spiel injectet und die adressen sind falsch, bekommt das spiel nen dc weil die dll auf irgendeine adresse zeigt etc.
ich hoffe das ich es besser erklärt hab, weil ich kein typ der alles 1a erklären kann

deswegen sag ich das seine adresse falsch ist

Siehe Punkt 1, das ist kein Overflow.


Man gibt keine Tipps, wenn man nicht weiß wovon man spricht.

[XxharCs]

ich zittiere einfach nur:

Man kann aber parameter auch zurück geben, aber nicht immer und nicht jeden ! (paar andere vorraussetzungen spielen da natürlich auch mit..)

vorraussetzung wär in dem fall das ich die länge zurück gebe(.length), ein weiteres beispiel ist wenn ich zb den parameter * 2 oder was anderes im return mache, ich hoffe du verstehst was ich meine
aber das weitere diskutieren lass ich mal erspart bleiben, willst du sicher auch nicht

nur noch beim:

Rückgabewert definiert, aber es wird nichts zurückgegeben.

hab ich deine aussage falsch intepretiert, nimms nicht böse, aber durch Sm!ths post hab ich dich schon verstanden was du meintest

Siehe Punkt 1, das ist kein Overflow.

als ich damals logs erstellt habe bei den probs die ich hatte, sagten die logs das es sich um einen overflow handelte da die adresse auf wichtige threads gezeigt hat und ich dann irgendwas verändert hab und deswegen stürzte es ab
aber Zugriffsverletzungen in diesem fall, würd ichs nicht nennen, da bei der zugriffsverletzung schon was anderes passiert

[2n0w]

Originally Posted by XxharCs

aber Zugriffsverletzungen in diesem fall, würd ichs nicht nennen, da bei der zugriffsverletzung schon was anderes passiert

Du kannst es nennen wie du willst, an einen "bad pointer" zu schreiben bringt dir wahrscheinlich eine Access violation ein.
Was, wenn nicht das hier, sollte sonst eine Access violation auslösen o.0 ?

[MrSm!th]

Originally Posted by XxharCs

ich zittiere einfach nur:

vorraussetzung wär in dem fall das ich die länge zurück gebe(.length), ein weiteres beispiel ist wenn ich zb den parameter * 2 oder was anderes im return mache, ich hoffe du verstehst was ich meine
aber das weitere diskutieren lass ich mal erspart bleiben, willst du sicher auch nicht

nur noch beim:

hab ich deine aussage falsch intepretiert, nimms nicht böse, aber durch Sm!ths post hab ich dich schon verstanden was du meintest


als ich damals logs erstellt habe bei den probs die ich hatte, sagten die logs das es sich um einen overflow handelte da die adresse auf wichtige threads gezeigt hat und ich dann irgendwas verändert hab und deswegen stürzte es ab
aber Zugriffsverletzungen in diesem fall, würd ichs nicht nennen, da bei der zugriffsverletzung schon was anderes passiert

Darauf lässt sich nur noch

Man gibt keine Tipps, wenn man nicht weiß wovon man spricht.

erwidern.