c - 我的程序意外停止运行，并且无法释放变量

Question

我编写了一个程序，可以在另一个文件中扫描一个文件的内容(以字节为单位)。它曾经工作得很好，但后来我了解到必须对输出进行排序，所以我添加了变量 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;
}

感谢您的帮助!

Answer 1

这里:

(*sortOrder)[i] = (char*)malloc((strlen(file) + ZERO_END) * sizeof(char));

您分配了足够的空间来容纳字符串文件，加上 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], " - ");

这显然是错误的。