非递归目录遍历函数随机崩溃问题排查求助

可能的核心原因 #

• 无效DIR指针或未初始化结构体：readdir依赖合法的DIR对象，如果opendir失败后仍继续使用空指针，或者DIR内部状态因重复使用未重置，会触发底层未初始化内存读取。
• 动态内存未初始化：FOLDER/ARCHIVE结构体分配后未初始化成员（比如路径缓冲区、内容指针），这些未初始化值被目录操作函数间接访问。
• Windows兼容层API冲突：栈 trace 显示readdir最终调用了Windows的_findnext64i32，说明使用的是POSIX兼容层实现，若代码中混用了不同风格的目录操作逻辑，会导致DIR结构体内部状态混乱。

具体解决步骤 #

1. 严格校验opendir返回值
   每次打开目录后必须检查是否成功，失败则立即终止当前处理，避免使用无效指针：

   DIR* dir = opendir(target_path);
   if (!dir) {
       perror("Failed to open directory");
       return; // 或根据逻辑处理错误后返回
   }
   

2. 正确管理DIR指针生命周期
   处理完一个目录后必须调用closedir释放资源，禁止复用未关闭的DIR指针处理其他目录：

   // 处理目录内容逻辑...
   closedir(dir);
   dir = NULL; // 显式置空，避免后续误操作
   

3. 初始化所有动态分配的结构体
   用memset或显式赋值初始化FOLDER/ARCHIVE的所有成员，避免未初始化内存被访问：

   FOLDER* new_folder = malloc(sizeof(FOLDER));
   if (new_folder) {
       memset(new_folder, 0, sizeof(FOLDER)); // 全置0初始化
       // 再设置具体字段值
   }
   

4. 排查遍历逻辑的边界问题
   非递归遍历用栈/队列存储待处理目录时，确保入队的路径是完全初始化的合法字符串，禁止传递未初始化的路径给opendir。

5. 可选：切换到Windows原生API
   若仅针对Windows平台开发，直接使用FindFirstFileW/FindNextFileW替代readdir，避免兼容层的潜在问题，同时更好支持Unicode路径（Minecraft资源包常含非ASCII字符）。

额外调试建议 #

• 开启编译器的内存检测选项：MSVC用/RTCu，GCC用-fsanitize=undefined，快速定位未初始化变量。
• 用DrMemory进一步跟踪：聚焦scanFolder函数中DIR指针的创建、复用，以及结构体内存分配的代码段，定位未初始化内存的具体来源。

内容的提问来源于stack exchange，提问作者Finn Demon Cat