我正在设计一个协议(在C中),使用cnet(http://www.csse.uwa.edu.au/cnet/)实现分层的OSI网络结构。我在运行时遇到SIGSEGV错误,但是cnet编译我自己的源代码文件(我无法通过gcc编译它)所以我不能轻易使用任何调试工具(如gdb)来查找错误。
以下是使用的结构和相关代码:
typedef struct {
char *data;
} DATA;
typedef struct {
CnetAddr src_addr;
CnetAddr dest_addr;
PACKET_TYPE type;
DATA data;
} Packet;
typedef struct {
int length;
int checksum;
Packet datagram;
} Frame;
static void keyboard(CnetEvent ev, CnetTimerID timer, CnetData data)
{
char line[80];
int length;
length = sizeof(line);
CHECK(CNET_read_keyboard((void *)line, (unsigned int *)&length)); // Reads input from keyboard
if(length > 1)
{ /* not just a blank line */
printf("\tsending %d bytes - \"%s\"\n", length, line);
application_downto_transport(1, line, &length);
}
}
void application_downto_transport(int link, char *msg, int *length)
{
transport_downto_network(link, msg, length);
}
void transport_downto_network(int link, char *msg, int *length)
{
Packet *p;
DATA *d;
p = (Packet *)malloc(sizeof(Packet));
d = (DATA *)malloc(sizeof(DATA));
d->data = msg;
p->data = *d;
network_downto_datalink(link, (void *)p, length);
}
void network_downto_datalink(int link, Packet *p, int *length)
{
Frame *f;
// Encapsulate datagram and checksum into a Frame.
f = (Frame *)malloc(sizeof(Frame));
f->checksum = CNET_crc32((unsigned char *)(p->data).data, *length); // Generate 32-bit CRC for the data.
f->datagram = *p;
f->length = sizeof(f);
//Pass Frame to the CNET physical layer to send Frame to the require link.
CHECK(CNET_write_physical(link, (void *)f, (size_t *)f->length));
free(p->data);
free(p);
free(f);
}
我设法找到了这一行:CHECK(CNET_write_physical(link,(void *)f,(size_t *)f-> length));引起了段错,但我找不到原因。 非常感谢任何帮助。
答案 0 :(得分:2)
我认为是第三个参数。试试这个:
CHECK(CNET_write_physical(link, (void *)f, (size_t *)(&f->length)));
在这一行中,我假设第三个参数需要一个指针,因为你要将值转换为(size_t *)
。但是您投射的值是一个简单的整数值。因此,每当函数取消引用该值中包含的地址时,就是您可能获得SIGSEGV。
使用我建议的代码,您正在投射指针(&f->length)
。所以你应该好好去,假设函数有效地期望指向一个持有大小的变量的指针。
答案 1 :(得分:0)
我在这里看到两个问题 - sizeof(f)
为您提供指针的大小而不是Frame
,然后您将size_t
- 类型值分配给{{1} },但稍后将其转换为f->length
。后者很可能是分割错误的原因。