Как сохранить бинарный файл из libzip

Моя цель - записать файл на диск

Если я использую gio для создания файла

GError *error;
char path[strlen(dirpath)]; 
sprintf(path, "%s", dirpath); // Create path to the file by copying from another variable
zip_file_t *contentfile = zip_fopen_index(book, index, ZIP_RDONLY);
zip_stat_index(book, index, ZIP_CHECKCONS, &fileinfo);
GFile *gfile = g_file_new_for_path(strcat(path, fileinfo.name));
GFileOutputStream *file = g_file_create(gfile, G_FILE_CREATE_NONE, g_cancellable_new(), &error);
if (error)
    printf("Error :%i\n", error->code);
else
    zip_fread(contentfile, file, fileinfo.size);
g_error_free(error);

Я не могу ничего вставить в файл. Это просто файл, созданный с 0 байтами. Я извлекаю все файлы из архива с помощью loo. Отсюда индекс в zip_file_t *contentfile = zip_fopen_index(book, index, ZIP_RDONLY);. Я получаю segfault: GError устанавливается поверх предыдущей GError или неинициализированной памяти.

Если я использую ФАЙЛ

char path[strlen(dirpath)]; 
sprintf(path, "%s", dirpath); // Create path to the file by copying from another variable
zip_file_t *contentfile = zip_fopen_index(book, index, ZIP_RDONLY);
zip_stat_index(book, index, ZIP_CHECKCONS, &fileinfo);
FILE *file = fopen(strcat(path, fileinfo.name), "wb");
zip_fread(contentfile, file, fileinfo.size);

Это дает ошибку сегментации: двойное освобождение или повреждение (out)

Как правильно записать содержимое zip_read( в файл на диске?


c libzip gio
person Bret Joseph    schedule 03.09.2020    source источник
comment
Для начала выделите достаточно памяти для path: измените ее на char path[strlen(dirpath) + strlen(fileinfo.name) + 1]; и переместите после вызовов zip_stat_index.   -  person MikeCAT    schedule 03.09.2020
comment
спасибо, я считаю, что это улучшает мой код, спасибо за совет   -  person Bret Joseph    schedule 03.09.2020

Ответы (1)


arrow_upward
0
arrow_downward

@MikeCAT был прав, мне сначала нужно было исправить размер пути к символу. Используя ту же идею, я изменил свой void *buffer на char buffer[size], а не на указатель. Он сохраняет файлы из некоторых почтовых индексов, некоторые из которых все еще дают ошибку сегментации. Я выясню, почему [возможно, где-то недостаточно памяти], но, по крайней мере, пока это работает. я обновлю позже

Пришлось установить буфер с помощью malloc, чтобы избежать ошибки, теперь это работает

static void extract_book(zip_t *book, char *bookname)
{
    int size = strlen(pubdir) + strlen(bookname) + 1;
    char pubpath[size];
    strncat(strncat(strcpy(pubpath, pubdir), bookname, strlen(bookname)), "/", 2);
    GFile *dir = g_file_new_for_path(pubpath);
    g_file_make_directory_with_parents(dir, NULL, NULL);
    for (int index = 0; index < zip_get_num_files(book); index++)
    {
        zip_file_t *contentfile = zip_fopen_index(book, index, 0);
        zip_stat_t fileinfo;
        int status = zip_stat_index(book, index, ZIP_FL_ENC_GUESS, &fileinfo);
        if (status == 0)
        {
            char *buffer = malloc(fileinfo.size);
            char filepath[size + strlen(fileinfo.name)];
            strcat(strcpy(filepath, pubpath), fileinfo.name);
            zip_fread(contentfile, buffer, fileinfo.size);
            zip_fclose(contentfile);
            FILE *file = fopen(filepath, "wb");
            fwrite(buffer, fileinfo.size, 1, file);
            fclose(file);
        }
        else
        {
            printf("%s at %i\n", puberror, zip_get_error(book)->zip_err);
        }
    }
}
person Bret Joseph    schedule 04.09.2020