C++ 尝试发送包含字符数组的数据包时出现分段错误（核心转储）

尝试发送包含最多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];
    }
  }
}

如果我将数据包中的数据内容设置为空，并将长度设置为0，则数据包可以很好地传输。一旦我试图增加数据包的长度并给它一个字符串，服务器就会遇到分段错误

---

我错过了什么？我觉得我的缓冲区太大了，或者可能我指向或引用了错误的内容。

您将接收数据包（

recvpckt

）初始化如下：

packet recvpckt(0, 0, 0, NULL);

这使得

数据

成员为空

然后调用

recvpckt.deserialize

，它将写入未初始化的

数据

数组。您需要在某处进行内存分配（例如在

recvpckt.deserialize

），或者将数组地址传递给

recvpckt

构造函数以将数据写入

---

这种问题最好使用调试器来解决。

这是C代码吗？看起来确实如此。一般来说，发布可编译代码是个好主意（请阅读如何创建MCVE-）。您有

std:string finalText它不编译。您不能将标签附加到变量定义上，AFAIK（当然不是在C中）和名称空间使用双冒号。@tadman:肯定有只使用C++的方面，比如范围运算符
：
和在代码中使用“未声明的变量”
this
。这标志着它是明确的，因为C++ + C++的C风格。是的，包也是一个类吗？我们赢了。这是C++的味道，我甚至没有注意到。这是在一些标准的图书馆爱的迫切需要。非常感谢你！我的教授警告我们，在尝试将数据反序列化到数据包中之前，需要初始化数据包的char*数据字段，但是我并不完全理解这意味着什么。非常感谢你指出这一点；现在，我可以将以零结尾的字符串传递给服务器@桑加克
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);

packet recvpckt(0, 0, 0, NULL);