C 编程 - 操作字符串数组时出现分段错误

Question

我正在努力解决当 dns 变量从大小 10 更改为大小 32 时发生的分段错误。我相信 main 中 dns ariable 的声明正在创建一个由 10 个字符串组成的数组，然后当增加到 32 个时，一个由 32 个字符串组成的数组。

因此，当使用大小为 10 的 dns 变量执行时，我得到了以字母 D 开头的 4 个目录名称的输出。这是预期的输出。

如果我将 dns 变量的大小增加到 32，我仍然会得到 4 个目录名称，然后立即出现分段错误。

自从我编码以来已经很长时间了，我确信我在某个地方违反了记忆。

非常感谢任何帮助。

谢谢，

丹。

void ld_dirs(char *strings[])
{
   int count;
   DIR *dp;
   struct dirent *ep;

   dp = opendir ("/tmp/sysinfo/inputs");
   if (dp != NULL)
   {
      count = 0;

      while (ep = readdir (dp))
      {
         if (ep->d_name[0] == 'D')
         {
            strings[count] = malloc (80 * sizeof(char));
            strings[count] = ep->d_name;
            count ++;
         }
      }

      closedir (dp);
   }
   else
      perror ("Could not open the directory");
}

int main ()
{
   char cwd[120];
   int count, count2;
   char *dns[10];
   char *path;
   if (getcwd(cwd, sizeof(cwd)) != NULL)
      printf ("\nCurrent working directory : %s\n\n\n", cwd);
   else
      perror ("getcwd() error");

   ld_dirs (dns);

   count = 0;

   puts ("List of valid inputs :\n");

   while (dns[count] != NULL)
   {
      puts (dns[count]);
      count ++;
   }     

   printf ("There are %d valid inputs\n", count);
   return 0;
}

Answer 1

我看到的问题：

1. 数组的大小。

   您已声明：

   char *dns[10];
   

   您在ld_dirs 中没有任何代码，以确保您不会使用strings 超出其合法持有的范围。

2. 您尚未将 dirs 中的项目初始化为 NULL，但您指望在以下 while 循环中将项目的值设为 NULL。

   while (dns[count] != NULL)
   {
      puts (dns[count]);
      count ++;
   }     
   

您可以通过以下方式解决这些问题：

1. 将dns中的项目初始化为NULL。

   char *dns[10] = {0};
   

2. 将数组的大小传递给ld_dir，并确保您不会使用超出法律规定的任何内容。

   函数界面改为：

   void ld_dirs(char *strings[], int size)
   

   将调用改为：

   ld_dirs (dns, sizeof(dns)/sizeof(*dns));
   

Answer 2

我马上看到的一个问题是您继续将readdir 的返回值用于不同的字符串表条目，并覆盖您刚刚分配用于保存名称的地址。但我不认为 readdir 保证为每次调用返回一个新的、分配的目录条目结构。我怀疑不会，访问之前的其中一个导致分段错误只是时间问题。

您需要从使用readdir 读取的每个目录条目中复制数据：

  while (ep = readdir (dp))
  {
     if (ep->d_name[0] == 'D')
     {
        strings[count] = malloc(80 * sizeof(char));
        //ERROR: strings[count] = ep->d_name;
        strcpy(strings[count], ep->d_name);  //CORRECTED
        count ++;
     }
  }

我没有检查超出这一点的其他错误。