C Lang - Struct Array和Segmentation fault(core dumped)

时间:2016-01-26 06:21:23

标签: c

我编写了一个简单的代码来读取目录,然后按上次修改时间对文件进行排序,但是下面的代码会崩溃,因为Segmentation fault(core dumped),(当尝试访问lsf [0]和lsf时发生) 1]),如何解决问题,请帮助,谢谢:

#include <stdio.h>
#include <stdlib.h>
#include <sys/types.h>
#include <dirent.h>
#include <sys/stat.h>
#include <time.h>
#include <string.h>

typedef struct {
  char *name;
  unsigned long lasted;
} sort_file;

char *make_path(char *path) {
  char *endp = path + strlen(path);
  *endp++ = '/';
  return endp;
}

int main(int arc, char **argv) {

  const char *n_dir = "/home/psycho/Pictures";
  char *dir_name = strdup(n_dir);

  char *endp = make_path(dir_name);

  DIR *dir;
  struct dirent *ent;
  struct stat attr;

  int size = 150;
  int count = 0;
  sort_file **lsf = malloc(size * sizeof(sort_file *));
  printf("[Logging] : %d => %d  \n", sizeof(sort_file), sizeof(sort_file *));
  if ((dir = opendir(n_dir)) != NULL) {
    while ((ent = readdir(dir)) != NULL) {

      if (ent->d_name[0] == '.') {
        continue;
      } else if (ent->d_type == DT_DIR) {
        strcpy(endp, ent->d_name);
        stat(dir_name, &attr);

        lsf[count] = malloc(sizeof(sort_file));
        lsf[count]->lasted = (unsigned long)attr.st_mtime;
        lsf[count]->name = malloc((strlen(ent->d_name) )* sizeof(char));

        sprintf(lsf[count]->name, "%s", ent->d_name);
        printf("[Logging] : --- %lu --- %d --- %s  \n", lsf[count]->lasted,
               count, lsf[count]->name);


        count++;

      }
    }
    closedir(dir);

  } else {
    return 0;
  }

  while (count--) {
    printf("[Logging] : --- %lu --- %d --- %s  \n", lsf[count]->lasted, count,
           lsf[count]->name);
  }
  return 0;
}

1 个答案:

答案 0 :(得分:2)

此代码包含一些内存恐慌,依赖于不同的编译器可能会出现不同的错误。

  1. make_path中,endp指向dir_name'\0')的最后一个字符,并尝试将其从'\0'更改为'/'无需在末尾插入字符串尾随字符('\0'),而realloc再添加一个字符dir_name
  2. main中,const char *n_dir = "/home/psycho/Pictures";不需要定义为指针,因为长度在编译时是已知的。实际上它应该改为:const char n_dir[] = "/home/psycho/Pictures";
  3. main中,endp根本没有分配,因此strcpy(endp, ent->d_name);副本到受限制的内存空间
  4. main中,lsf[count]->name = malloc((strlen(ent->d_name) )* sizeof(char));应该再分配一个单元,因为尾随'\0'lsf[count]->name = malloc((strlen(ent->d_name) + 1 )* sizeof(char));
  5. 最后我发现了一个逻辑错误: if (ent->d_name[0] == '.') { ...让程序跳过“”,“ .. ”,以及以“”开头的所有其他目录。 '(Linux隐藏文件夹)。您可以使用0 == (strcmp(ent->d_name, ".") * strcmp(ent->d_name, ".."))