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;

（我们的想法是会有一个单独的链接列表＆＃34; jobs＆＃34;多个线程将处理，每个线程都包含该作业的单个链接文件列表。）

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()加入到我的files_list结构中。然后在步骤＃2和步骤＃3的前几行之间发生了一些事情，它将文件名截断为24个字符并添加＆＃34; 1。＆＃34;指针地址表明它仍然是相同的结构对象，我没有看到任何可能弄乱它的操作。

我使用过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，并且您必须检查几乎所有函数的返回值有一个的函数，因为它是确保函数工作且没有意外发生的唯一方法，否则为什么会有返回值呢？

奇怪的Char *截断

1 个答案: