libzip无法关闭文件

时间:2014-07-14 14:56:04

标签: c++ c++11 libzip

我目前在C ++ 11程序中使用libzip来提取压缩文件的内容并将它们存储到一个数据结构中,该数据结构还将保存与该文件相关的元数据。

我使用当前方法来展开zip文件并获取其中每个文件的内容:

void explodeArchive(const string& path, vector<ZipFileModel>& files) {
    int error = 0;
    zip *zip = zip_open(path.c_str(), 0, &error);
    if (zip == nullptr) {
      throw logic_error("Could not extract content of file " + path);
    }
    const zip_int64_t n_entries = zip_get_num_entries(zip, ZIP_FL_UNCHANGED);
    for (zip_int64_t i = 0; i < n_entries; i++) {
      const char *file_name = zip_get_name(zip, i, ZIP_FL_ENC_GUESS);
      struct zip_stat st;
      zip_stat_init(&st);
      zip_stat(zip, file_name, ZIP_FL_NOCASE, &st);
      char *content = new char[st.size];
      std::cerr << file_name << std::endl;
      zip_file *file = zip_fopen(zip, file_name, ZIP_FL_NOCASE);
      const zip_int64_t did_read = zip_fread(file, content, st.size);
      if (did_read <= 0) {
        continue;
      }
      if (strlen(content) < st.size) {
        LOG(WARNING)<< "File " << file_name << " is truncated.";
      }
      if (strlen(content) > st.size) {
        content[st.size] = '\0';
      }

      ZipFileModel model;
      model.name = string(file_name);
      model.content = string(content);
      model.order = -1;
      files.push_back(model);

      zip_fclose(file);
      delete[] content;
    }
    zip_close(zip);
  }

我的问题是我得到随机分段错误,gdb指向zip_fclose(file);

Program received signal SIGSEGV, Segmentation fault.
0x00000001001ef8a0 in zip_source_close (src=0x105001b00) at /Users/xxx/Projects/xxx/xxx/src/libzip/zip_source_close.c:48
48      (void)src->cb.l(src->src, src->ud, NULL, 0, ZIP_SOURCE_CLOSE);

调试此问题的最佳方法是什么?正如我所说,它间歇性地发生,所以很难确定原因。

1 个答案:

答案 0 :(得分:1)

当没有什么可读的时候,你没有关闭zip_file。

首先打开里面的文件:

zip_file *file = zip_fopen(zip, file_name, ZIP_FL_NOCASE);

然后尝试阅读:

  const zip_int64_t did_read = zip_fread(file, content, st.size);

如果没有什么可读的,那么继续,文件永远不会关闭。

  if (did_read <= 0) {
    continue;
  }

所以,只需添加:

if (did_read <= 0) {
    zip_fclose(file);
    continue;
}