Отображение содержимого папки в c - Ошибка времени выполнения

Question

Я пытаюсь показать, какая папка содержит в качестве вывода. Когда я запускаю эту программу на своем жестком диске через 1-2 минуты, она падает, а также сбойная часть работает нормально. Я не знаю, как я могу это предотвратить. Может кто-нибудь мне помочь ?

#include <string.h> 
#include <stdio.h> 
#include <dirent.h> 

void showingFiles(DIR *, char *); 

int main(void) { 
    DIR *folder; 
    char path[350]; 

    sprintf(path, "."); 
    folder = opendir(path); 
    showingFiles(folder, path); 
    closedir(folder); 
    printf("\n\nEnter a key to close this program "); 
    getch(); 
    return 0; 
} 

void showingFiles(DIR *currentFolder, char *path){ 
    struct dirent *nextFile; 
    DIR *subFolder; 
    char copyPath[350]; 
    strcpy(copyPath, path); 
    while ((nextFile = readdir(currentFolder)) != NULL) { 
     sprintf(copyPath, "%s//%s", path, nextFile->d_name); 
     printf("%s\n", (*nextFile).d_name); 
     if ((strcmp(nextFile->d_name, "..")) && 
      strcmp(nextFile->d_name ,".") && 
      (subFolder = opendir(copyPath)) != NULL) { 
      deletingFiles(subFolder, copyPath); 
     } 
    } 
    closedir(currentFolder); 
} 

Answer 1

Есть по крайней мере 3 проблемы в вашем коде, которые могут объяснить крах:

• буферов, используемые для хранения полных имен путей платить слишком короткие, и вы используете небезопасную sprintf построить их, потенциально вызывая переполнение буфера.
• Вы никогда не закрываете ручки каталога subFolder, которые вы открываете в рекурсивной функции showingFiles, потенциально заканчивая системными ручками.
• вы закрываете ручку каталога currentFolder в функции showingFiles(), но она также закрыта в функции main(). Это вызывает неопределенное поведение. Как правило, всегда закрывайте дескриптор функции, которая открывала ее и только там.

Менее важные, но вопросы:

• Чтобы назвать showingFiles функцию, которая выполняет рекурсивный удаление полного дерева каталогов немного вводит в заблуждение.

• разделяющий каталог и пути с двойными косыми чертами // бесполезен и не переносится. Возможно, вы думали о \\ и преобразовали этот разделитель разделов Windows в // для портативности Unix, но имейте в виду, что одиночные косые черты :, поддерживаемые обработчиками файловой системы Windows, всегда должны использовать / в качестве разделителя каталогов для программ предназначенный как для Unix, так и для Windows.

Вот модифицированная версия:

#include <dirent.h> 
#include <stdio.h> 
#include <string.h> 

void deleteTree(DIR *, const char *); 

int main(void) { 
    char path[350] = "."; 
    DIR *folder = opendir(path); 

    if (folder != NULL) { 
     deleteTree(folder, path); 
     closedir(folder); 
    } 
    printf("\n\nEnter a key to close this program "); 
    getch(); 
    return 0; 
} 

void deleteTree(DIR *currentFolder, const char *path) { 
    char copyPath[1024]; 
    struct dirent *nextFile; 
    DIR *subFolder; 

    while ((nextFile = readdir(currentFolder)) != NULL) { 
     snprintf(copyPath, sizeof(copyPath), "%s/%s", path, nextFile->d_name); 
     printf("%s\n", nextFile->d_name); 
     if (strcmp(nextFile->d_name,"..") 
     && strcmp(nextFile->d_name,".") 
     && (subFolder = opendir(copyPath)) != NULL) { 
      deletingFiles(subFolder, copyPath); 
      closedir(subFolder); 
     } 
    } 
}