[C++] Hilfe zu c++/cli restore bytes

[boss8513]

Solved, Ty <3

[Jeoni]

Verstehe das Problem nicht ganz. Wenn du die Originalbyte wiederherstellen möchtest, kopier sie halt vorher in ein eigenes Puffer, bevor du sie mit NOPs überschreibst. Dann kannst du jederzeit die Originaldaten aus deinem Puffer wieder an ihren korrekten Platz kopieren.
Ferner schlägt dein zweiter VirtualProtect-Aufruf in WriteToMemory zwingend fehl, weil du als letztes Argument NULL übergibst, s. .

Viele Grüße

[boss8513]

Originally Posted by Jeoni

Verstehe das Problem nicht ganz. Wenn du die Originalbyte wiederherstellen möchtest, kopier sie halt vorher in ein eigenes Puffer, bevor du sie mit NOPs überschreibst. Dann kannst du jederzeit die Originaldaten aus deinem Puffer wieder an ihren korrekten Platz kopieren.
Ferner schlägt dein zweiter VirtualProtect-Aufruf in WriteToMemory zwingend fehl, weil du als letztes Argument NULL übergibst, s. .

Viele Grüße

Dies habe ich versucht, leider gibt er mir immer 00 bytes, daher denke ich, ich mache etwas falsch.

Habe versucht es über die Writetomemory Funktion umzuschreiben.

char testbytes[6];

void CopyToMemory(uintptr_t addressToStore, void const* valueToRead, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToStore), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToStore, valueToRead, byteNum);
    VirtualProtect((LPVOID)(addressToStore), byteNum, OldProtection, &OldProtection);
}

Danach rufe ich dies vor dem nopen aus.

CopyToMemory(test, &testbytes, 6);

und nach dem ich die Checkbox unchecke -> 

WriteToMemory(test, testbytes, 6);

Replacet er einfach die 6 bytes mit 6 00ern. ^^

[Jeoni]

Du hast Quell- und Ziel-Pointer vertauscht. Statt von der Originaladresse in den Puffer kopierst du von deinem Puffer zur Adresse, s. .

Viele Grüße

[boss8513]

Originally Posted by Jeoni

Du hast Quell- und Ziel-Pointer vertauscht. Statt von der Originaladresse in den Puffer kopierst du von deinem Puffer zur Adresse, s. .

Viele Grüße

void CopyToMemory(void* addressToStore, void const* valueToRead, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToStore), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToStore, valueToRead, byteNum);
    VirtualProtect((LPVOID)(addressToStore), byteNum, OldProtection, &OldProtection);
}

CopyToMemory(&testbytes, &test, 6);

Habe es nun so probiert, Problem ist, er schreibt nun irgendwelche Random Bytes rein und nicht die "Default" Bytes.

[Omdi]

Originally Posted by boss8513

void CopyToMemory(void* addressToStore, void const* valueToRead, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToStore), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToStore, valueToRead, byteNum);
    VirtualProtect((LPVOID)(addressToStore), byteNum, OldProtection, &OldProtection);
}

CopyToMemory(&testbytes, &test, 6);

Habe es nun so probiert, Problem ist, er schreibt nun irgendwelche Random Bytes rein und nicht die "Default" Bytes.

void CopyToMemory(void* addressToStore, void const* valueToRead, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToStore), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToStore, valueToRead, byteNum);
    VirtualProtect((LPVOID)(addressToStore), byteNum, OldProtection, &OldProtection);
}

bool TestMyFunc()
{
    unsigned int dummy = 0x12345678;
    unsigned int _i_need_to_be_dummy = 0;
    CopyToMemory(&_i_need_to_be_dummy, &dummy, sizeof(dummy));
    return _i_need_to_be_dummy == dummy;
}

if (!TestMyFunc())
{
    // nope i did something wrong
}

[boss8513]

Originally Posted by Omdi

void CopyToMemory(void* addressToStore, void const* valueToRead, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToStore), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToStore, valueToRead, byteNum);
    VirtualProtect((LPVOID)(addressToStore), byteNum, OldProtection, &OldProtection);
}

bool TestMyFunc()
{
    unsigned int dummy = 0x12345678;
    unsigned int _i_need_to_be_dummy = 0;
    CopyToMemory(&_i_need_to_be_dummy, &dummy, sizeof(dummy));
    return _i_need_to_be_dummy == dummy;
}

if (!TestMyFunc())
{
    // nope i did something wrong
}

Habe ich 1:1 so probiert, resultat von &dummy sowie &_i_need_to_be_dummy über cout
ergeben leider aber wieder nicht die richtigen bytes :/

sie ergeben die Addresse die sie in meiner DLL haben, gehe ich nun dahin um zugucken ob sie diese bytes dort übernommen haben, haben sie nur 2 bytes die auch nichts damit zutun haben.

[Omdi]

Originally Posted by boss8513

Habe ich 1:1 so probiert, resultat von &dummy sowie &_i_need_to_be_dummy über cout
ergeben leider aber wieder nicht die richtigen bytes :/

Es macht auch keinen Sinn die beiden Adressen zu vergleichen.
Poste mal deinen kompletten Code, du machst wahrscheinlich nur irgendwo einen recht dummen Fehler.

[boss8513]

Originally Posted by Omdi

Es macht auch keinen Sinn die beiden Adressen zu vergleichen.
Poste mal deinen kompletten Code, du machst wahrscheinlich nur irgendwo einen recht dummen Fehler.

 Spoiler

void WriteToMemory(uintptr_t addressToWrite, void const* valueToWrite, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToWrite), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToWrite, valueToWrite, byteNum);
    VirtualProtect((LPVOID)(addressToWrite), byteNum, OldProtection, &OldProtection);
}

void CopyToMemory(void* addressToStore, void const* valueToRead, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToStore), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToStore, valueToRead, byteNum);
    VirtualProtect((LPVOID)(addressToStore), byteNum, OldProtection, &OldProtection);
}

char nopbytes[] = "\x90\x90\x90\x90\x90\x90";
DWORD test = FindPattern(testmodule, testsig, testmask);

unsigned int dummy = test;
unsigned int _i_need_to_be_dummy = 0;

bool TestMyFunc()
{
	CopyToMemory(&_i_need_to_be_dummy, &dummy, sizeof(dummy));
	return _i_need_to_be_dummy == dummy;
}


private: System::Void checkBox1_CheckedChanged(System::Object^ sender, System::EventArgs^ e) {

		if (checkBox1->Checked == true) 

		{
			TestMyFunc(); <- habe dies auch in form1_load getestet, um 
                                         es dann mit cout auszulesen.
			WriteToMemory(test, nopbytes, 6);
		}
		else
		{
			WriteToMemory(test, &_i_need_to_be_dummy, 6);

		}
	}

Glaube das müsste der ganze code sein zu diesem zweck.

[Omdi]

Müsste so gehen:

void WriteToMemory(uintptr_t addressToWrite, void const* valueToWrite, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToWrite), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToWrite, valueToWrite, byteNum);
    VirtualProtect((LPVOID)(addressToWrite), byteNum, OldProtection, &OldProtection);
}

char origbytes[6] = {0};
DWORD test = FindPattern(testmodule, testsig, testmask);
if (test != 0)
{
	WriteToMemory((uintptr_t)origbytes, test, 6);
}

private: System::Void checkBox1_CheckedChanged(System::Object^ sender, System::EventArgs^ e) {

		if (checkBox1->Checked == true) 
		{
			char nopbytes[] = "\x90\x90\x90\x90\x90\x90";
			WriteToMemory(test, nopbytes, 6);
		}
		else
		{
			WriteToMemory(test, origbytes, 6);

		}
	}

[boss8513]

Originally Posted by Omdi

Müsste so gehen:

void WriteToMemory(uintptr_t addressToWrite, void const* valueToWrite, int byteNum)
{
    unsigned long OldProtection;
    VirtualProtect((LPVOID)(addressToWrite), byteNum, PAGE_EXECUTE_READWRITE, &OldProtection);
    memcpy((LPVOID)addressToWrite, valueToWrite, byteNum);
    VirtualProtect((LPVOID)(addressToWrite), byteNum, OldProtection, &OldProtection);
}

char origbytes[6] = {0};
DWORD test = FindPattern(testmodule, testsig, testmask);
if (test != 0)
{
	WriteToMemory((uintptr_t)origbytes, test, 6);
}

private: System::Void checkBox1_CheckedChanged(System::Object^ sender, System::EventArgs^ e) {

		if (checkBox1->Checked == true) 
		{
			char nopbytes[] = "\x90\x90\x90\x90\x90\x90";
			WriteToMemory(test, nopbytes, 6);
		}
		else
		{
			WriteToMemory(test, origbytes, 6);

		}
	}

Hat geklappt, musste zwar test von dword noch konvertieren, da er nicht mit void will, aber es geht nun!

WriteToMemory((uintptr_t)&origbytes, (uint32_t*)test, 6);

Danke euch beiden für die hilfe .