C - 指向struct中的动态数组的指针"分段错误(核心转储)"

时间:2015-02-22 03:16:08

标签: c arrays pointers dynamic struct

我正在编写ac程序来从目录中读取文件和目录,然后指出在结构数据中找到的元素数量,并将动态数组中元素的名称指向同一结构的数据中。我做到了,它的输出是正确的。问题是当我运行程序时出现“分段故障(核心转储)”。

代码:

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

#define EXIT_FAILURE 1

typedef struct FileDir
{   
    int *n_files;
    char *file_name[];
} FileDir;

int get_files_within_dir(struct FileDir *fd)
{
    DIR *dir;
    int n_files;
    int index;

    n_files = 0;
  if ((dir = opendir ("/tmp")) != NULL) {
    /* counts all the files and directories within directory */
    while (readdir (dir) != NULL) {
      n_files++;
    }
    closedir (dir);
  } else {
    /* could not open directory */
    perror ("");
    return EXIT_FAILURE;
  }       

  char *file_name[n_files];
  struct dirent *ent;
  if ((dir = opendir ("/tmp")) != NULL) {
    /* gets all the files and directories within directory */
    index = 0;
    while ((ent = readdir (dir)) != NULL) {
      file_name[index++] = ent->d_name;      
    }
    closedir (dir);
  } else {
    /* could not open directory */
    perror ("");
    return EXIT_FAILURE;
  }      

  fd->n_files = n_files;  
  fd->file_name[0] = file_name[0];
  fd->file_name[1] = file_name[1];
  fd->file_name[2] = file_name[2];
  fd->file_name[3] = file_name[3];

  return 0;  
}

int main()
{   
    struct FileDir fd;
    get_files_within_dir(&fd);
    printf("%d\n", fd.n_files);
    printf("%s\n", fd.file_name[1]);
    printf("%s\n", fd.file_name[2]);
    printf("%s\n", fd.file_name[3]);    

    return 0;
}

输出:

[freitas@localhost src]$ ./file_dir 
21
..
geany_socket.fcda02b3
tmpiSdUX3
Segmentation fault (core dumped)

有趣的是,如果我只是将小于或等于2的值指向结构数据的动态数组,则错误消息不会显示出来。你有什么想法吗?

谢谢!

2 个答案:

答案 0 :(得分:0)

您有2个问题,可能导致SEGMENTATION FAULT

  1. n_files字段是一个指针,你为它分配了一个整数,它应该被声明为

    int n_files;
    
  2. 您无法为file_name字段分配空间,您至少应提供固定尺寸,例如

    char *file_name[1000];
    

    您可以使用malloc()动态分配内存,但这是另一回事,需要解释。

  3. 注意:启用编译器警告可以帮助您防止int *n_files之类的愚蠢错误,然后执行fd->n_files = n_files;

答案 1 :(得分:0)

n_files不应该是指针

typedef struct FileDir
{   
 int n_files;
 char *file_name[];
} FileDir;

然后你的行

 printf("%d\n", fd.n_files);

不会崩溃。尝试使用调试器

查看结构