1

次のようなパケット構造があります。

struct packet {
    sockaddr_in m_Recv;
    int m_iPacketSize;
    unsigned char* m_ucpPacket;
}

これは、この構造を使用しているコードの選択です。

packet tOutPacket;
tOutPacket.m_Recv = tInPacket.m_Recv;
tOutPacket.m_iPacketSize = 256;
tOutPacket.m_ucpPacket = new unsigned char[256];

// Creating a uuid (not important how it's done, the pointer has 16 bytes)
unsigned char* uuid = NetworkHandlerN::ConnectionData::createUUID(hardware_hash);
NetworkHandlerN::PacketHandler::addUUIDToPacket(tOutPacket.m_ucpPacket, uuid);
delete[] uuid;

UDPsend(tOutPacket.m_Recv, tOutPacket.m_ucpPacket, tOutPacket.m_iPacketSize);

/* Delete pointer */
delete[] tOutPacket.m_ucpPacket; // This is causing an error

これが私のUDPsendメソッドです：

bool NBsocket::UDPsend(sockaddr_in& AddrSento, const unsigned char* cucpBuffer, int iSize)
{
    if(sendto(m_iSocket, reinterpret_cast<const char*>(cucpBuffer), iSize, 0, (sockaddr*) &AddrSento, sizeof(AddrSento)) == -1) {
        m_iErrorCode = 6;
        return false;
    }

    return true;
}

addUUIDToPacket メソッドは次のとおりです。

static void addUUIDToPacket(unsigned char* ucpPacket, unsigned char* ucpChar) 
{
    for(int i = 0; i < ciUUIDSize; i++) {
        ucpPacket[ciHeaderSize+i] = ucpChar[i];
    }
}

tOutPacket.m_ucpPacket最初はどこかでポインターを削除したと思ったのですが、それを確認しました。そして、このように内部に保存されているデータに対してテストを実行すると(の直前delete[]):

if(tOutPacket.m_ucpPacket[tOutPacket.m_iPacketSize-1] == 0x08) 
    std::cout << "0x08";

それは完全に正常に動作します（はい、値は0x08であるはずです）

ここに私のコールスタック出力があります:

    ntdll.dll!77dcfadc()    Unknown
    [Frames below may be incorrect and/or missing, no symbols loaded for ntdll.dll] 
    ntdll.dll!77db4f92()    Unknown
    ntdll.dll!77d926fc()    Unknown
    ntdll.dll!77dd0b37()    Unknown
    ntdll.dll!77d8a967()    Unknown
>   msvcp110.dll!std::_Fputc<char>(char _Byte, _iobuf * _File) Line 83  C++
    msvcp110.dll!std::basic_filebuf<char,std::char_traits<char> >::overflow(int _Meta) Line 383 C++
    msvcp110.dll!std::basic_streambuf<char,std::char_traits<char> >::xsputn(const char * _Ptr, __int64 _Count) Line 406 C++

すでに多くのことをテストしましたが、何が問題なのかわかりませんでした。

誰かが私を助けてくれることを願っています。必要に応じてパーツを追加できます/* add data to packet */が、それにはもう少しコードが必要です。

編集

エラーが見つかりました。このメソッドにバイトを渡しました：

static void addToPacket(unsigned char* ucpPacket, int iInt, int iPos) 
{
    ucpPacket[iPos+0] = iInt & 0xFF;
    ucpPacket[iPos+1] = iInt >> 8 & 0xFF;
    ucpPacket[iPos+2] = iInt >> 8 & 0xFF;
    ucpPacket[iPos+3] = iInt >> 8 & 0xFF;
}

これにより、すべての問題が発生しました...メソッドのオーバーロードが多すぎます:)。概要を失ったに違いありません。

4

1 に答える 1

0

バグとその原因を実際に見つける前に、new[]とdelete[]を使用する必要があるかどうかを検討してください。

  1. サイズは256に固定されていますか？構造体に通常の配列を入れることができますか？それほど大きくはありません。

  2. そうでなければ、使用できますstd::vector<unsigned char>か？

  3. 失敗すると使用できますboost::shared_array<unsigned char>か？サイズも別の要素として入力する必要があります（例：iPacketSize）。このためにあなたはまだ電話をしますnew[]が、delete[]世話をします。shared_array使用するメモリが必要な限り、このオブジェクトを保持する必要があります。

何をしcreateUUIDますか？それは戻ってきますunsigned char *が、誰がこの記憶を所有していますか？

送信したデータはどの時点でコピーされますか？電話でsendto？

于 2012-10-23T15:22:32.977 に答える