c - 无法使用结构来存储和稍后使用 recvfrom 打印 udp 数据
问题描述
我正在尝试使用以下结构将收到的数据放入:
typedef struct packet {
uint8_t magic;
uint8_t version;
uint16_t body_length;
char *body;
} packet;
这是打印功能:
void displayPacket (struct packet p){
printf("Magic : %u\n",p.magic);
printf("Version : %u\n",p.version);
printf("Body Length: %d\n",ntohs(p.body_length));
printf("Body : %s\n",p.body);
}
在我的主要工作中,我正在尝试将接收到的数据保存在我的结构中:
unsigned char reply[1024];
struct packet reply_packet;
reply_packet.body = malloc(1021);
reply_packet.body[1020] = '\0';
rc = recvfrom(s,&reply_packet,sizeof(reply_packet),0,NULL,NULL);
displayPacket(reply_packet);
free(reply_packet.body);
close(s);
输出:
Magic : 95
Version : 1
Body Length: 1008
[1] 4741 segmentation fault ./network
Magic、Version、Body Length 是数据包开头的预期输出。由于其他对等方遵循的协议,数据包的最大限制大小为 1024 字节。
但是,我的 displayPacket 函数引起了分段错误,更准确地说是这一行:
printf("Body : %s\n",p.body);
如果可以提供帮助,这是 valgrind 输出:
==4886== Invalid read of size 1
==4886== at 0x4C32CF2: strlen (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==4886== by 0x4E994D2: vfprintf (vfprintf.c:1643)
==4886== by 0x4EA0F25: printf (printf.c:33)
==4886== by 0x108EFD: displayPacket (network.c:143)
==4886== by 0x1090FF: main (network.c:189)
==4886== Address 0x3030000000000000 is not stack'd, malloc'd or (recently) free'd
==4886==
==4886==
==4886== Process terminating with default action of signal 11 (SIGSEGV)
==4886== General Protection Fault
==4886== at 0x4C32CF2: strlen (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==4886== by 0x4E994D2: vfprintf (vfprintf.c:1643)
==4886== by 0x4EA0F25: printf (printf.c:33)
==4886== by 0x108EFD: displayPacket (network.c:143)
==4886== by 0x1090FF: main (network.c:189)
尽管它看起来很微不足道,但它一直困扰着我几个小时......
预期的输出将是数据包中其余内容的内容。到目前为止,我已经能够做到。
请问您有什么解决方案来解决这个问题?
谢谢。
编辑 1:
通过像这样修改结构解决了分段错误问题:
typedef struct packet {
uint8_t magic;
uint8_t version;
uint16_t body_length;
char body[1021];
} packet;
和其余的代码:
unsigned char reply[1024];
struct packet reply_packet;
reply_packet.body[1020] = '\0';
rc = recvfrom(s,&reply_packet,sizeof(reply_packet),0,NULL,NULL);
printf("body[0] %d\n",reply_packet.body[0]);
displayPacket(reply_packet);
close(s);
displayPacket 保持不变。
解决方案
packet.body 是一个指针。传递指针实际上没有意义,因为远程机器上的数据地址是无用的,而且您将始终只读取主体的 sizeof(pointer) 字节。你可以这样做:
typedef struct packet {
uint8_t magic;
uint8_t version;
uint16_t body_length;
char body[MAX_BODY_SIZE];
} packet;
您在 printf 中崩溃,因为您正在取消引用错误的指针。即使 body 具有可变长度,您也可以将 MAX_BODY_SIZE 设置为可能的最大尺寸,然后使用 body_length 字段来计算实际尺寸。
你也可以去掉 reply 变量,因为它没有被使用。
更新:你很接近。您正在将正文打印为数字。既然你提到了 ASCII,我假设它是一个字符串。此外,您需要根据正文的实际长度终止 NULL。这是您需要的:
struct packet reply_packet;
rc = recvfrom(s,&reply_packet,sizeof(reply_packet),0,NULL,NULL);
reply_packet.body[packet.body_length] = '\0'; // null term after the body length
printf("body[0] %s\n",reply_packet.body); // change to %s and remove [0], you want the pointer not a character
displayPacket(reply_packet);
推荐阅读
- three.js - Three.js 在多台计算机的服务器上分发高分辨率渲染
- gluon - 集成到项目自定义reflectionconfig-arm64-ios
- javascript - 滚动上的jQuery动画不会向左/向右移动元素
- android - 调试版本中未调用 Application.onCreate()
- r - 将模式添加到 r 中的箱线图
- c++ - 在 C++ 中更优雅地打破循环引用
- angular - Angular - 使用辅助路由生成 routerLink
- machine-learning - 我应该先执行交叉验证,然后再进行网格搜索吗?
- html - HTML 更改图像大小
- sql-server - 将自动生成的 id 传递给其他表