Моя программа неожиданно останавливается и не может освободить переменную - PullRequest
Новая
       5

Моя программа неожиданно останавливается и не может освободить переменную

0 голосов
/ 23 мая 2018

Я создал программу, которая сканирует содержимое файла (в байтах) в другом файле.Раньше он работал хорошо, но потом я узнал, что вывод должен быть отсортирован, поэтому я добавил переменную sortOrder и немного изменил свою программу, чтобы она сохраняла данные, а не просто распечатывала их.Проблема в том, что теперь во время работы моя программа сталкивается с ошибкой в ​​случайных точках.Другая проблема, с которой я столкнулся, заключается в том, что по какой-то причине я не могу освободить строки внутри sortOrder в main, он просто возвращает ошибку.Я почти уверен, что проблема либо в 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;
}

Спасибо за помощь!

1 Ответ

0 голосов
/ 24 мая 2018

Здесь: 

(*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], " - ");

, что, очевидно, неправильно.

...