Question

我编写了一个简单的代码来读取目录，然后按上次修改时间对文件进行排序，但是下面的代码会崩溃，因为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;
}

Answer 1

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

在make_path中，endp指向dir_name（'\0'）的最后一个字符，并尝试将其从'\0'更改为'/'无需在末尾插入字符串尾随字符（'\0'），而realloc再添加一个字符dir_name
在main中，const char *n_dir = "/home/psycho/Pictures";不需要定义为指针，因为长度在编译时是已知的。实际上它应该改为：const char n_dir[] = "/home/psycho/Pictures";
在main中，endp根本没有分配，因此strcpy(endp, ent->d_name);副本到受限制的内存空间
在main中，lsf[count]->name = malloc((strlen(ent->d_name) )* sizeof(char));应该再分配一个单元，因为尾随'\0'：lsf[count]->name = malloc((strlen(ent->d_name) + 1 )* sizeof(char));

最后我发现了一个逻辑错误： if (ent->d_name[0] == '.') { ...让程序跳过“。”，“ .. ”，以及以“”开头的所有其他目录。 '（Linux隐藏文件夹）。您可以使用0 == (strcmp(ent->d_name, ".") * strcmp(ent->d_name, ".."))

C Lang - Struct Array和Segmentation fault（core dumped）

1 个答案: