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

Quote:
Originally Posted by MartPwnS ^
Some updates from the WoM2 side:
So hier mal ein paar Neuigkeiten von WoM2:

[For developers mostly | Haupts�chlich f�r Entwickler]
Spoiler
Only pay closer attention to the second vulnerability if the following is true for you.
Die zweite L�cke l�sst sich nur unter der folgenden Bedingung ausnutzen.
Code:
check_name    != check_name_alphabet;
A nicer way to fix the original vulnerability and even save some queries (YAAYYYY!) is:
Eine bessere L�sung um die Originall�cke zu stopfen und um ein paar Abfragen zu sparen ist die folgende:

Search for the following text in MessengerManager::RemoveFromList:
Sucht nach dem folgenden Text in ^
Code:
if (companion.size() == 0)
    return;
and add the following lines below of that:
und f�gt die folgenden Zeilen danach ein:
Code:
if (!IsInList(account, companion))
    return;
Then add the following function to messenger_manager.cpp:
Jetzt k�nnt ihr die folgende Funktion in der ^ hinzuf�gen:
Code:
bool MessengerManager::IsInList(MessengerManager::keyA account, MessengerManager::keyA companion)
{
    if (m_Relation.find(account) == m_Relation.end())
        return false;

    if (m_Relation[account].empty())
        return false;

    return m_Relation[account].find(companion) != m_Relation[account].end();
}
and ofc dont forget to declare it in messenger_manager.h:
und nat�rlich auch nicht vergessen diese in ^ zu deklarieren:
Code:
bool    IsInList(MessengerManager::keyA account, MessengerManager::keyA companion);
PS: Der Original fix ist nicht schlecht lediglich etwas am Ziel vorbei (funktionieren tut er auch)
PS: The original fix is not bad its just a little bit over the top (it works perfectly fine )

Kind regards
Mit freundlichen Gr��en
MartPwnS for WoM2

Bitte um Hilfe

Ist die messenger_manager.h so richtig?

PHP Code:


			
#ifndef __INC_MESSENGER_MANAGER_H

#define __INC_MESSENGER_MANAGER_H



#include "db.h"



class MessengerManager : public singleton<MessengerManager>

{

    public:

        typedef std::string keyT;

        typedef const std::string & keyA;



        MessengerManager();

        virtual ~MessengerManager();



    public:

        void    P2PLogin(keyA account);

        void    P2PLogout(keyA account);



        void    Login(keyA account);

        void    Logout(keyA account);



        void    RequestToAdd(LPCHARACTER ch, LPCHARACTER target);

        void    AuthToAdd(keyA account, keyA companion, bool bDeny);



        void    __AddToList(keyA account, keyA companion);    // 실제 m_Relation, m_InverseRelation 수정하는 메소드

        void    AddToList(keyA account, keyA companion);



        void    __RemoveFromList(keyA account, keyA companion); // 실제 m_Relation, m_InverseRelation 수정하는 메소드

        void    RemoveFromList(keyA account, keyA companion);    



        void    RemoveAllList(keyA account);



        void    Initialize();



    private:

        void    SendList(keyA account);

        void    SendLogin(keyA account, keyA companion);

        void    SendLogout(keyA account, keyA companion);



        void    LoadList(SQLMsg * pmsg);



        void    Destroy();



        std::set<keyT>            m_set_loginAccount;

        std::map<keyT, std::set<keyT> >    m_Relation;

        std::map<keyT, std::set<keyT> >    m_InverseRelation;

        std::set<DWORD>            m_set_requestToAdd;

        

        bool    IsInList(MessengerManager::keyA account, MessengerManager::keyA companion);

};



#endif