奇怪的字符 * 截断

Question

我刚刚回到 C 编程领域，但我遇到了一个奇怪的问题，即结构中的动态 char* 字段。它一开始完全没问题，而且长度合适，但它被截断为 25 个字符，其中最后一个字符甚至不在原始数组中。

我定义了两个结构和几个全局变量：

struct files_list {
  struct files_list* next;
  char* fpath;
};

struct job_list {
  int len;
  off_t size;
  struct files_list* files;
  struct job_list* next;
};

struct job_list* job_head = NULL;
struct job_list* active = NULL;

（这个想法是会有一个“作业”的链接列表，多个线程将处理每个由该作业的单个文件链接列表组成。）

Main 只是调用ntfw 来遍历目录树：

int main(int argc, char** argv) {
  // ...
  nftw("/tmp/", populate, 100, 0);
  // ...
}

对于每个文件和目录，ntfw 都会调用我的populate 函数：

int populate(const char *fpath, const struct stat *sb,
             int tflag, struct FTW *ftwbuf) {



  if (tflag == FTW_F) {
    /* #1 CREATE NEW FILE ENTRY */
    printf("\n\n");
    printf("FPATH: %s\n", fpath);
    struct files_list* current_files = malloc(sizeof(struct files_list));
    current_files->fpath = malloc(sizeof(fpath) + 1);
    strcpy(current_files->fpath, fpath);
    current_files->next = NULL;
    printf("files_list %s (%i)\n", current_files->fpath, strlen(current_files->fpath)); 
    printf("&current_files: %p\n", current_files);



    /* #2 CREATE NEW ACTIVE JOB */
    if (active == NULL) {
      struct job_list* job = malloc(sizeof(struct job_list));
      job->len = 0;
      job->size = 0;
      job->files = NULL;
      job->next = NULL;
      active = job;
    }
    active->len++;
    active->size += sb->st_size;


    /* #3 INSERT FILE ENTRY AT THE END OF ACTIVE JOB */
    if (active->files == NULL) {
      active->files = current_files;
      printf("&active->files: %p\n", active->files);
      printf("current_files->fpath: %s (%i)\n", current_files->fpath, strlen(current_files->fpath)); // TRUNCATED STRING
      printf("active->files->fpath: %s (%i)\n", active->files->fpath, strlen(active->files->fpath));
    } else {                                                                                                                                                                                               
      struct files_list* x = active->files->next;                                                                                                                                                                  
      while (1) {                                                                                                                                                                                                  
        if (x == NULL) {                                                                                                                                                                                           
          x = current_files;                                                                                                                                                                                       
          printf("Added files_list: %s\n", x->fpath);                                                                                                                                                              
          break;                                                                                                                                                                                                   
        } else {                                                                                                                                                                                                   
          printf("next x->next\n");                                                                                                                                                                                
          x = x->next;                                                                                                                                                                                             
        }                                                                                                                                                                                                          
      }
    }

    /* #4 TIME FOR A NEW JOB */
    if (active->size > MAX_BYTES) {
      if (job_head == NULL) {
        job_head = active;
        job_tail = active;
      } else {
        job_tail->next = active;
      }
      struct job_list* new_active = NULL;
      active = new_active;
      job_len++;
    }
  }
  return 0;
}

产生这样的输出：

FPATH: /tmp/.com.google.Chrome.T8MuMk/SingletonSocket
files_list /tmp/.com.google.Chrome.T8MuMk/SingletonSocket (46)
&current_files: 0x125d1e0
&active->files: 0x125d1e0
current_files->fpath: /tmp/.com.google.Chrome.1 (25)
active->files->fpath: /tmp/.com.google.Chrome.1 (25)

从前两行我们可以看出，文件路径很长，包含 46 个字符，并且成功地将strcpy()ed 到我的files_list 结构中。然后在第 2 步和第 3 步的前几行之间发生了一些事情，将文件名截断为 24 个字符并添加了“1”。指针地址表明它仍然是同一个结构对象，并且我看不到任何可能与它混淆的操作。

我用过 GCC 4.9.2 和 Clang 3.5.0，他们都有这个问题，所以一定是我做错了。

有什么想法吗？

Answer 1

这就是问题

malloc(sizeof(fpath) + 1);

sizeof(fpath) 将给出一个指针的大小，因为fpath 是一个char *，即一个char 指针，你需要strlen()

malloc(strlen(fpath) + 1);

不要忘记包含string.h。

您需要了解sizeof 运算符，这是来自 1570 草案，§ 6.5.3.4 段落 ¶ 2。

sizeof 运算符产生其操作数的大小（以字节为单位），可能是 表达式或类型的括号名称。大小由类型决定 操作数。结果是一个整数。如果操作数的类型是变长数组类型，则计算操作数；否则，不计算操作数，结果为整数常量。

注意：你必须检查malloc()函数族的返回值，当失败发生时它们返回NULL，你必须检查几乎每个函数的返回值，因为只有这样才能保证函数正常工作，没有意外发生，不然怎么会有返回值呢？