c - 我的程序意外停止运行,并且无法释放变量
问题描述
我制作了一个程序来扫描另一个文件中的文件内容(以字节为单位)。它曾经运行良好,但后来我了解到必须对输出进行排序,所以我添加了变量 sortOrder,并稍微更改了我的程序,以便它保存数据而不是仅仅打印出来。问题是现在我的程序在运行时随机点出现错误。我遇到的另一个问题是,由于某种原因,我无法在 main 中释放 sortOrder 中的字符串,它只会返回一个错误。我很确定问题出在 main 或 statusOfFiles 中,尽管我可能是错的,因为我并不真正理解这个问题。
#define _CRT_SECURE_NO_WARNINGS
#include <errno.h>
#include <stdio.h>
#include <string.h>
#include <dirent.h>
#include <stdlib.h>
#include <sys/stat.h>
#define ADD_NAME 1
#define MAX_LEN 200
#define ZERO_END 1
#define VIRUS 2
#define CLEAN 0
#define INFECTED 1
#define FIRST_PART 2
#define LAST_PART 3
void addResults(int foundDetails, char*** sortOrder, int i)
{
if (foundDetails == CLEAN)
{
strcat((*sortOrder)[i], "Clean");
}
else if (foundDetails == INFECTED)
{
strcat((*sortOrder)[i], "Infected!");
}
else if (foundDetails == FIRST_PART)
{
strcat((*sortOrder)[i], "Infected! (first 20%%)");
}
else
{
strcat((*sortOrder)[i], "Infected! (last 20%%)");
}
}
void statusOfFiles(char * filePath, char ** files, int lenFiles, char*** sortOrder)
{
char file[MAX_LEN] = { 0 };
int i = 0;
for (i = 0; i < lenFiles; i++)
{
strcpy(file, filePath);
strcat(file, "/");
strcat(file, files[i]);
*sortOrder = (char**)realloc((*sortOrder), (i + ADD_NAME) * sizeof(char*));
(*sortOrder)[i] = (char*)malloc((strlen(file) + ZERO_END) * sizeof(char));
strcat(strcpy((*sortOrder)[i], file), " - ");
addResults(0, sortOrder, i);
}
}
void printResultsInOrder(char** sortOrder, int len)
{
int i = 0;
for (i = 0; i < len; i++)
{
printf("%s\n", sortOrder[i]);
}
}
int main(void)
{
int i = 0, len = 5;
char *fileNames[] = {"adgv.txt", "fsagda", "adagda", "adga", "adgastgaet"}, **sortOrder = (char**)malloc(0);
statusOfFiles("C:/folder", fileNames, len, &sortOrder);
printResultsInOrder(sortOrder, len);
for (i = 0; i < len; i++)
{
free(sortOrder[i]);
}
free(sortOrder);
getchar();
return 0;
}
谢谢您的帮助!
解决方案
这里:
(*sortOrder)[i] = (char*)malloc((strlen(file) + ZERO_END) * sizeof(char));
你 malloc 足够的空间来保存字符串文件加上 1 的终止空值。下一行:
strcat(strcpy((*sortOrder)[i], file), " - ");
您将文件复制到其中(完全填充空间),然后在其末尾添加“ - ”,超出缓冲区。
一个可能会帮助您自己找到这一点的样式说明:将尽可能多的代码打包到一行中会使代码难以阅读,并且现代编译器不会降低速度。我会把它写成
(*sortOrder)[i] = malloc((strlen(file) + ZERO_END) * sizeof(char));
strcpy((*sortOrder)[i], file);
strcat((*sortOrder)[i], " - ");
这使错误更加明显。实际上,因为 strdup() 为你做了分配和复制,我可能把它写成
(*sortOrder)[i] = strdup(file);
strcat((*sortOrder)[i], " - ");
这显然是错误的。
推荐阅读
- html - 空白:nowrap;css 样式不允许表格响应
- html - HTML5 border not fitting in to margin-right
- java - 在 apache kafka 主题上流式传输没有输出
- javascript - 带有昵称和头像的公会成员更新日志频道
- java - 如何在 Spring 运行时关闭接收器 bean
- c++ - 指向 C++ 寄存器的指针是否合法?
- r - 将“Ymw”转换为日期 - 一周的第一个星期一
- ssl - 我正在尝试将 ESP8266 与 HTTPS python 服务器连接,但 ESP 未与服务器连接并给出服务器名称错误的 Bearssl 错误代码
- c# - ASP.NET Core appsettings.json reloadOnChange 不适用于符号链接文件?
- python - 如何让我的机器人发送图像以及如何创建关闭它的命令?