Question

我一直试图调试一段时间，仍然无法弄清楚为什么会导致堆栈粉碎错误（我认为错误代码是6，或者是中止。本质上这个函数需要一个目录，打开一个文件，然后将该文件放入一个函数中，以便它可以使用该文件，然后输出它通过该函数的次数。

int map(char* dir, void* results, size_t size, int (*act)(FILE* f, void* res, char* fn))
 {
      printf("%s\n", dir);
      char copyDirectory[strlen(dir)+1];
      //adds the slash
      strcpy(copyDirectory, dir);
      strcat(copyDirectory, "/");
      //before the psuedocode, get all the files in the directory
      int numFiles = nfiles(copyDirectory);
      DIR* directory = opendir(copyDirectory);
      //if there aren't any files, then we exit
      if(numFiles == 0)
      {
           closedir(directory);
           return -1;
      }
      //reads the file from the directory
      struct dirent* readFile = readdir(directory);
      int output = 0;
      while(readFile!=NULL)
      {
           if(readFile->d_type==DT_REG)
           {
               //step 2: obtain filepath
               char* fileName = readFile->d_name;
               int filePathLength = strlen(dir) + strlen(fileName) + 1;//add one for the slash
               char filePath[filePathLength];
               memset(filePath, 0, filePathLength); //allocat ememory for file path
               strcpy(filePath, strcat(dir, fileName));
               //step 3: open file
               FILE* file = fopen(filePath, "r");
               //if the file is unreachable, exit
               if(file==NULL)
               {
                    closedir(directory);
                    return -1;
               }
               //step 4: perform some action and store result
               strcpy(dir, copyDirectory);
               act(file, results, fileName);
               //step 5: close file
               fclose(file);
               //to go through loop: increment the readFile
                    ++output;
                }
                readFile = readdir(directory);
      }
      closedir(directory);
      return output;
 }

使用示例映射函数。

int map(char* dir, void* results, size_t size, int (*act)(FILE* f, void* res, char* fn))
{
     char* copyDirectory = strdup(dir);
     DIR* directory = opendir(dir);
     int output = 0;
     struct dirent* readFile = readdir(directory);
     while(readFile!=NULL)
     {
         if(readFile->d_type==DT_REG)
         {
             //step 2: obtain filepath
             char* fileName = readFile->d_name;
             int filePathLength = strlen(dir) + strlen(fileName) +2;//add one for the slash
             char filePath[filePathLength+1];
             memset(filePath, 0, filePathLength); //allocat ememory for file path
             strcpy(filePath, strcat(dir, fileName));
             //step 3: open file
             FILE* file = fopen(filePath, "r");
             //if the file is unreachable, exit
             if(file==NULL)
             {
                 closedir(directory);
                 return -1;
             }
             //step 4: perform some action and store result
             strcpy(dir, copyDirectory);
             act(file, results, fileName);
             //step 5: close file
             fclose(file);
             //to go through loop: increment the readFile
             ++output;
         }
         readFile = readdir(directory);
     }  
     closedir(directory);
     return output;
}
//Sample Map function action: Print file contents to stdout and returns the number bytes in the file.
int cat(FILE* f, void* res, char* filename) {
    char c;
    int n = 0;
    printf("%s\n", filename);
    while((c = fgetc(f)) != EOF) {
        printf("%c", c);
        n++;
    }
    printf("\n");
    return n;
}
int main(int argc, char const *argv[])
{
    char directory[]= "../rsrc/ana_light/";
    size_t size = 100;
    void* results[500]; 
    int mapCat = map(directory, results, size, cat);
    printf("The value of map is %d.\n", mapCat);
    return EXIT_SUCCESS;
}

失败的地方是执行完毕后打印到输出。该函数应打印出您拥有的文件的内容。目录列表需要有一个＆＃34; /＆＃34;在末尾。目前它打印文件内容并以其读取的文件的数量退出，但它在退出时会因堆栈粉碎错误而退出。

EDIT1：编辑代码以反映我所做的更改。

EDIT2：我认为根据MCVE标准完成了吗？如果我没有弄错的话应该跑。

Answer 1

第一个问题：改变

__init__.py

到

    char copyDirectory[strlen(dir)+1];

第二个问题：改变

    char copyDirectory[strlen(dir)+2];

到

       char filePath[filePathLength];

第三个问题（第一次阅读时似乎没有改变）：

       char filePath[filePathLength+1];

注释掉的代码是正确的：

     //strcpy(copyDirectory, dir);
     strcat(copyDirectory, dir);

您已经忘记了跟踪空字符的空格。

第四个问题：你忘了处理opendir失败了。

第五个问题：这段代码错了：

    strcpy(copyDirectory, dir);

更改为：

        memset(filePath, 0, filePathLength); //allocat ememory for file path
        strcpy(filePath, strcat(dir, fileName));

请勿在此处写入输入变量。这是一个非常糟糕的主意。

在撤消copyDirectory的（漏洞）strdup并将你非常创新的本地缓冲区重新放回之后，我能够让代码运行完成。

使用strcpy和strcat时堆栈粉碎

1 个答案: