2

最大 30 文字のパケットを他の識別情報と共に送信しようとしています。ここにコンストラクタがあります。

packet::packet(int t, int s, int l, char * d){
type = t;
seqnum = s;
length = l;
data = d;
}

私のクラスでは、次の方法でシリアル化および逆シリアル化する必要があります。

void packet::serialize(char * spacket){
sprintf (spacket, "%d %d %d %s", type, seqnum, length, data);   
}

void packet::deserialize(char * spacket){
char * itr;
itr = strtok(spacket," ");
char * null_end;

this->type = strtol(itr, &null_end, 10);

itr = strtok(NULL, " ");
this->seqnum = strtol (itr, &null_end, 10);

itr = strtok(NULL, " ");
this->length = strtol (itr, &null_end, 10);

if(this->length == 0){
    data = NULL;
}
else{
    itr = strtok(NULL, " ");    
    for(int i=0; i < this->length; i++){ // copy data into char array
        this->data[i] = itr[i];
    }
  }
}

このクライアント コードにより、server.exe でセグメンテーション エラーが発生します。

char sendPayload[512];
char receivePayload[512];
char sendBuffer[30] = {'1', '2', '3', '\0'};
int receivedPacketType = 0;
int sendPacket;
int receivePacket;

bzero(receivePayload, 512);
bzero(sendPayload, 512);

for (int iterator = 0; iterator < 8; iterator++)
{
    //Setting up sendPayload for sending.
    packet sendpckt(1, iterator, strlen(sendBuffer), sendBuffer);
    sendpckt.serialize((char*)sendPayload);

    //Datagram send to emulator.
    sendPacket = sendto(sendSocket, &sendPayload, sizeof(sendPayload), 0, (struct sockaddr *)&sendSocketStruct, sendSocketLen);

    //Datagram wait on receive from emulator.
    receivePacket = recvfrom(receiveSocket, &receivePayload, sizeof(sendPayload), 0, (struct sockaddr *)&receiveSocketStruct, &receiveSocketLen);

    //Setting up packet for receivePayload.
    packet recvpckt(0,0,0,NULL);
    recvpckt.deserialize((char*)receivePayload);
    recvpckt.printContents();

    bzero(receivePayload, 512);
    bzero(receivePayload, 512);

}

サーバー側の受信コードは次のとおりです。

char receivePayload[512];
char sendPayload[512];
int expectedSeqNum = 0;
int receivedPacketType = 0;
int receivedPacketSeqNum = 0;
int receivedPacketLength = 0;
char receivedPacketData[512];
int receivePacket;
int sendPacket;
std:string finalText;

bzero(receivePayload, 512);
bzero(sendPayload, 512);

do
{
    receivePacket = recvfrom(receiveSocket, &receivePayload, sizeof(receivePayload), 0, (struct sockaddr *)&receiveSocketStruct, &receiveSocketLen);
    packet recvpckt(0, 0, 0, NULL);
    recvpckt.deserialize((char*)receivePayload);
    receivedPacketType = recvpckt.getType();
    receivedPacketSeqNum = recvpckt.getSeqNum();
    receivedPacketLength = recvpckt.getLength();
    bzero(receivePayload, 512);
    recvpckt.printContents();

    if (receivedPacketType == 3)
    {
        break;
    }

    /*
    if (receivedPacketType == 1)
    {
        *receivedPacketData = *recvpckt->getData();
        printf("%s\n", receivedPacketData);
    }
    */

    if (receivedPacketSeqNum == expectedSeqNum)
    {
        packet sendpckt(0, expectedSeqNum, 0, NULL);
        sendpckt.serialize((char*)sendPayload);
        sendPacket = sendto(sendSocket, &sendPayload, sizeof(sendPayload), 0, (struct sockaddr *)&sendSocketStruct, sendSocketLen);
        bzero(sendPayload, 512);
        expectedSeqNum = expectedSeqNum + 1;
        if (expectedSeqNum > 7)
        {
            expectedSeqNum = 0;
        }

    }
    else
    {
        packet sendpckt(0, expectedSeqNum, 0, NULL);
        sendpckt.serialize((char*)sendPayload);
        sendPacket = sendto(sendSocket, &sendPayload, sizeof(sendPayload), 0, (struct sockaddr *)&sendSocketStruct, sendSocketLen);
        bzero(sendPayload, 512);
    }

} while (1);

パケット内のデータの内容を null にして長さを 0 に設定すると、パケットは完全に正常に転送されます。パケットの長さを増やして文字列を指定しようとするとすぐに、サーバーでセグメンテーション エラーが発生します。

ここで何が欠けていますか？バッファーを大きくしすぎているか、間違ったものを指し示したり参照したりしているように感じます。

4

1 に答える 1