Cipher Handshake Core Crash Fix

  • Konuyu açan Konuyu açan sauber
  • Açılış Tarihi Açılış Tarihi
  • Yanıt Yanıt 0
  • Gösterim Gösterim 96

sauber

Üye
Üye
Mesaj
106
Çözümler
2
Beğeni
89
Puan
699
Ticaret Puanı
0
Selamlar,

Aktif sunucuda yaşadım, nasıl yapıldığını yazmıyorum başka sunucular patlatılmasın diye özetle CG_HANDSHAKE ile client DH2KeyAgreement oluştuktan sonra tekrar aynı paket gönderildiğinde duplike ediliyor, debug buildde burada hata durumu assert ettirilmiş, yani debug buildde bunu server hata olarak görüp crash oluyor fakat release buildde bu kontrol yok, bu sebeple duplike olduktan sonra cleanup olduğunda memory leak olmakta.

cipher.cpp


Arat:
C++:
Genişlet Daralt Kopyala
size_t Cipher::Prepare(void* buffer, size_t* length)

metodu komple değiştir:
C++:
Genişlet Daralt Kopyala
size_t Cipher::Prepare(void* buffer, size_t* length) {
  // Prevent double initialization - memory corruption guard
  if (key_agreement_ != NULL) {
    sys_err("Cipher::Prepare() called while key_agreement_ already exists - potential double initialization");
    delete key_agreement_;
    key_agreement_ = NULL;
  }

  key_agreement_ = new DH2KeyAgreement();
  if (key_agreement_ == NULL) {
    sys_err("Failed to allocate DH2KeyAgreement");
    return 0;
  }

  size_t agreed_length = key_agreement_->Prepare(buffer, length);
  if (agreed_length == 0) {
    delete key_agreement_;
    key_agreement_ = NULL;
  }
  return agreed_length;
}

input.cpp

Arat:
C++:
Genişlet Daralt Kopyala
void CInputProcessor::Handshake(LPDESC d, const char * c_pData)

Komple değiştir:
C++:
Genişlet Daralt Kopyala
void CInputProcessor::Handshake(LPDESC d, const char * c_pData)
{
    TPacketCGHandshake * p = (TPacketCGHandshake *) c_pData;

    if (d->GetHandshake() != p->dwHandshake)
    {
        sys_err("Invalid Handshake on %d", d->GetSocket());
        d->SetPhase(PHASE_CLOSE);
    }
    else
    {
        if (d->IsPhase(PHASE_HANDSHAKE))
        {
            if (d->HandshakeProcess(p->dwTime, p->lDelta, false))
            {
#ifdef _IMPROVED_PACKET_ENCRYPTION_
                // Prevent duplicate key agreement if already prepared
                if (!d->IsCipherPrepared())
                {
                    d->SendKeyAgreement();
                }
                else
                {
                    sys_err("Duplicate handshake attempt on socket %d - cipher already prepared", d->GetSocket());
                    d->SetPhase(PHASE_CLOSE);
                }
#else
                if (g_bAuthServer)
                    d->SetPhase(PHASE_AUTH);
                else
                    d->SetPhase(PHASE_LOGIN);
#endif // #ifdef _IMPROVED_PACKET_ENCRYPTION_
            }
        }
        else
            d->HandshakeProcess(p->dwTime, p->lDelta, true);
    }
}



Değişiklikler aşağıdaki gibi olacaktır:

159197 eklentisini görüntüle

159196 eklentisini görüntüle
 
En son bir moderatör tarafından düzenlenmiş:
Geri
Üst