如何在c中正确使用scandir()？

Question

我试图在char**变量中存储文件列表.

scandir()完成正常,但在尝试打印char**时出现分段错误.

这是代码:

int main()
{
    char** fileList;
    int noOfFiles;
    char* path = ".";
    makeList(&fileList, &noOfFiles, path); 
    return 0;
}

void makeList(char ***fileList, int* noOfFiles, char* path){
    struct dirent **fileListTemp;
    *noOfFiles = scandir(path, &fileListTemp, NULL, alphasort);
    int i;
    fileList = (char***)malloc(sizeof(char***));
    *fileList = (char**)malloc(*noOfFiles * sizeof(char*));
    printf("total: %d files\n",*noOfFiles);
    for(i = 0; i < *noOfFiles; i++){
        *fileList[i] = (char*)malloc(strlen(fileListTemp[i] -> d_name) *sizeof(char));
        strcpy(*fileList[i], fileListTemp[i] -> d_name);
        printf("%s\n",*fileList[i]);
    }
    return;
}

打印2个文件名后,这会出现分段错误.

输出:

总计:27个档案.

..

.jv

分段故障(核心转储)

Answer 1

该函数scandir()为您分配内存.

您不需要分配任何内存.你不要需要释放的内存返还给您scandir().

您的代码调用: *noOfFiles = scandir(path, &fileListTemp, NULL, alphasort);

返回时,noOfFiles将包含目录中的目录条目数path,并fileListTemp指向已分配字符串的已分配字符串,每个字符串指向包含文件的以空值终止的名称的已分配块.

如果您的目录包含文件"FirstFile.txt","AnotherFile.txt","ThirdFile.txt",例如,您的电话,在返回时scandir(),noOfFiles将为这三个文件设置为5,另外两个用于" ".和".."目录条目.如果您不通过'alphasort',那么参赛作品将不会有任何特定的订单.(实际上这有点不正确.它们将按目录文件名条目的顺序排列,这取决于最初创建文件的顺序.)

因为您传递了'alphasort',您应该按以下顺序看到条目(我明确地显示了null-byte-string-terminator:

fileListTemp[0] == ".\0"
fileListTemp[1] == "..\0"
fileListTemp[2] == "AnotherFile.txt\0"
fileListTemp[3] == "FirstFile.txt\0"
fileListTemp[4] == "ThirdFile.txt\0"

所以fileListTemp指向一个包含五个(char*)指针的已分配内存块.五个(char*)指针中的每一个都指向一个包含空终止目录条目名称的已分配内存块.

那是分配内存的SIX块.

您可以使用此分配的内存,直到完成它为止,然后在数组中的EACH条目上调用free(),然后调用数组本身的free().

你必须释放每个条目以及数组本身.它们都是独立分配的内存块.

完成列表后,您应该:

for (int i = 0; i < noOfFiles; i++)
  {
  free(fileListTemp[i];
  }

free(fileListTemp);

Answer 2

这是一个老问题,但是由于我遇到它并且它没有像手册页那样有效地解决我的问题,我正在从手册页复制代码片段作为未来的新答案.

  #include <dirent.h>

   int
   main(void)
   {
       struct dirent **namelist;
       int n;

       n = scandir(".", &namelist, NULL, alphasort);
       if (n < 0)
           perror("scandir");
       else {
           while (n--) {
               printf("%s\n", namelist[n]->d_name);
               free(namelist[n]);
           }
           free(namelist);
       }
   }