Ошибка сегментации (дамп ядра) при попытке отправить пакет, содержащий массив символов

Попытка отправить пакет, содержащий до 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);

Если я обнуляю содержимое данных в пакете и устанавливаю длину в 0, пакеты передаются совершенно нормально. Как только я пытаюсь увеличить длину пакета и дать ему строку, сервер испытывает ошибку сегментации.

Что мне здесь не хватает? Я чувствую, что делаю свои буферы слишком большими или, может быть, я указываю или ссылаюсь на что-то неправильно.

2

Решение

Вы инициализируете свой пакет приема (recvpckt) как это:

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

Что делает data член NULL.

Тогда вы звоните recvpckt.deserialize, который пишет в неинициализированный data массив. Вам нужно где-то выделить память (например, в recvpckt.deserialize) или передать адрес массива recvpckt конструктор для записи данных в.

Такую проблему лучше всего решить с помощью отладчика.

1

Другие решения

Других решений пока нет …

По вопросам рекламы [email protected]