C++ - Sql injection-Arkadaş-Fısıltı-Lonca Fix

ONURSTEEL · 1 Şub 2017

Bilinen ve doğabilecek tüm "Sql İnjection" fixi ;

MAREK38' Alıntı:

../Source/game/src/db.cpp aç

Arat ;

Kod:

void DBManager::Query(const char * c_pszFormat, ...)

Kod bloğunu değiştir ;

Kod:

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());
}

Arat ;

Kod:

SQLMsg * DBManager::DirectQuery(const char * c_pszFormat, ...)

Değiştir ;

Kod:

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());
}

Başlıktaki gibi Sql injection-Arkadaş-Fısıltı-Lonca Fixi

messenger_manager.cpp aç

Arat ;

Kod:

void MessengerManager::RemoveFromList(MessengerManager::keyA account, MessengerManager::keyA companion)

Kod bloğunu değiştir ;

Kod:

void MessengerManager::RemoveFromList(MessengerManager::keyA account, MessengerManager::keyA companion)
{
    if (companion.empty())
        return;


    char companionEscaped[CHARACTER_NAME_MAX_LEN * 2 + 1];
    DBManager::instance().EscapeString(companionEscaped, sizeof(companionEscaped), companion.c_str(), companion.length());




    DBManager::instance().Query("DELETE FROM messenger_list%s WHERE account='%s' AND companion = '%s'",
                                get_table_postfix(), account.c_str(), companionEscaped);




    __RemoveFromList(account, companion);


    sys_log(1, "Messenger Remove %s %s", account.c_str(), companion.c_str());


    TPacketGGMessenger pack;
    pack.bHeader = HEADER_GG_MESSENGER_REMOVE;
    strlcpy(pack.szAccount, account.c_str(), sizeof(pack.szAccount));
    strlcpy(pack.szCompanion, companion.c_str(), sizeof(pack.szCompanion));
    P2P_MANAGER::instance().Send(&pack, sizeof(TPacketGGMessenger));
}

guild_manager.cpp aç

Arat ;

Kod:

std::unique_ptr<SQLMsg> pmsg(DBManager::instance().DirectQuery("SELECT COUNT(*) FROM guild%s WHERE name = '%s'",
                                                                   get_table_postfix(), __escape_name));
Üstteki kod yoksa bunu arat ;


    std::auto_ptr<SQLMsg> pmsg(DBManager::instance().DirectQuery("SELECT COUNT(*) FROM guild%s WHERE name = '%s'",
                get_table_postfix(), gcp.name));

Üstüne ekle ;

Kod:

    static char __escape_name[GUILD_NAME_MAX_LEN * 2 + 1];
        DBManager::instance().EscapeString(__escape_name, sizeof(__escape_name), static_cast<const char *>(gcp.name), sizeof(gcp.name));

Var ise arat ;

Kod:

    get_table_postfix(), gcp.name

Değiştir ;

Kod:

    get_table_postfix(), __escape_name

RENARD · 1 Şub 2017

Bir sene kadar sonradan gelıyorsun ama olsun

Edit//

Ek olarak sadece db.cpp'den bir kod değiştirerek bilinen ve doğabilecek tüm sql injectleri fixlemiş olursun.

merhametmt2 · 1 Şub 2017

bizlerle paylaşırsan seviniriz kardeş teşekkürler iyi forumlar

RENARD' Alıntı:
Bir sene kadar sonradan gelıyorsun ama olsun

Edit//

Ek olarak sadece db.cpp'den bir kod değiştirerek bilinen ve doğabilecek tüm sql injectleri fixlemiş olursun.

Black Widow · 1 Şub 2017

Konudaki gereksiz yorumlar silindi.

Lütfen bahsi geçen buglar ile ilgili açıklama veya video&fotoğraf şeklinde kanıt ekleyiniz.

AzadBaraN · 1 Şub 2017

RENARD' Alıntı:
Bir sene kadar sonradan gelıyorsun ama olsun

Edit//

Ek olarak sadece db.cpp'den bir kod değiştirerek bilinen ve doğabilecek tüm sql injectleri fixlemiş olursun.

Bİlgi paylaşıldıkça güzeldir. Paylaş o zaman sende şu SQL olayı tamamıyla kapansın ?

C++ - Sql injection-Arkadaş-Fısıltı-Lonca Fix

ONURSTEEL

Level 5

RENARD

Level 4

merhametmt2

Level 4

Black Widow

Level 5

AzadBaraN

Level 4

Şu an konuyu görüntüleyenler (Toplam : 0, Üye: 0, Misafir: 0)

Benzer konular

C++ - Sql injection-Arkadaş-Fısıltı-Lonca Fix

ONURSTEEL

Level 5

RENARD

Level 4

merhametmt2

Level 4

Black Widow

Level 5

AzadBaraN

Level 4

Şu an konuyu görüntüleyenler (Toplam : 0, Üye: 0, Misafir: 0)

Benzer konular

Privacy & Transparency

Privacy & Transparency