在C语言中如何创建包含其他类型结构体指针的链表节点
问题原因分析
你代码的核心错误是未给Frame结构体中的字符串指针分配内存就直接执行字符串拷贝,另外还有两处不严谨的实现也可能引发运行错误:
- 字符串野指针访问:
Frame结构体定义的char* name和char* path只是字符串指针,你调用malloc(sizeof(Frame))仅分配了存储两个指针和duration的内存,两个指针本身未指向合法的可写内存空间,直接调用strcpy相当于向随机地址写入数据,会触发内存访问错误。 - 缺少内存分配校验:两次
malloc调用都未检查返回值是否为NULL,如果内存分配失败,后续的指针操作会直接触发空指针崩溃。 - 链表节点未初始化完整:新创建的
FrameNode的next成员是野指针,后续遍历链表到该节点时会访问非法内存。
修正后代码实现
#include <stdlib.h> #include <string.h> /** * 创建新的帧节点 * 输入:name 帧名称,duration 帧持续时长,path 帧资源路径 * 输出:创建成功返回新节点指针,失败返回NULL */ FrameNode* createFrame(const char* name, unsigned int duration, const char* path) { // 分配链表节点内存 FrameNode* newFrame = malloc(sizeof(FrameNode)); if (!newFrame) { return NULL; } // 分配Frame结构体内存 newFrame->frame = malloc(sizeof(Frame)); if (!newFrame->frame) { free(newFrame); return NULL; } // 分配name内存并拷贝 size_t name_len = strlen(name) + 1; newFrame->frame->name = malloc(name_len); if (!newFrame->frame->name) { free(newFrame->frame); free(newFrame); return NULL; } strcpy(newFrame->frame->name, name); // 分配path内存并拷贝 size_t path_len = strlen(path) + 1; newFrame->frame->path = malloc(path_len); if (!newFrame->frame->path) { free(newFrame->frame->name); free(newFrame->frame); free(newFrame); return NULL; } strcpy(newFrame->frame->path, path); // 赋值duration,初始化next指针 newFrame->frame->duration = duration; newFrame->next = NULL; return newFrame; }
如果你的编译环境支持POSIX标准的strdup函数,可以简化字符串分配+拷贝的逻辑,替换为:
newFrame->frame->name = strdup(name); newFrame->frame->path = strdup(path);
替换后依然需要检查返回值是否为NULL,避免内存分配失败。
内容的提问来源于stack exchange,提问作者RaphDaPingu
相关产品推荐
相关产品推荐

