Копирование содержимого из одного файла в другой, ошибка сегментации

Question

Я пытаюсь скопировать 50 jpegs, один за другим из большого файла .raw, однако в настоящее время я получаю ошибку ошибки сегментации. Вот мой код:

#include <stdio.h> 
#include <stdlib.h> 
#include <stdint.h> 

typedef uint8_t BYTE; 

//SOI - 0xFF 0xD8 
//EOI - 0xFF 0xD9 
//APPn - 0xFF 0xEn  
int main(void) 
{ 
    //FAT - 512 bytes per block 
    BYTE block[512]; 

    //open file containing pictures 
    FILE* card_file = fopen("card.raw", "rd"); 

    FILE* jpeg_file; 

    //make sure the file opened without errors 
    if (card_file == NULL) 
    { 
     printf("something went wrong and file could not be opened"); 
     return 1; 
    } 

    int i = 0; 
    while (fread(&block, sizeof(BYTE), 512, card_file) != 0) 
    { 
     //jpeg start signature 
     if(block[0] == 0xFF && block[1] == 0xD8) 
     { 
      i++; 
      if(jpeg_file != NULL) 
       fclose(jpeg_file); 

      //create a new jpeg file to copy bytes to 
      jpeg_file = fopen((char*)i, "w+"); 
     } 

     //write 512 bytes to a jpeg file 
     if(jpeg_file != NULL) 
      fwrite(block, sizeof(block), 1, jpeg_file); 
    } 

    fclose(card_file); 
return 0; 
} 

, когда я запускаю его через GDB, мой код получает весь путь, если (блок [0] == 0xFF & & блока 1 == 0xD8), то он пропускает состояние и сегментацию происходит ошибка. Я не понимаю, что может быть причиной этого. Вот скриншот:

Код обновление:

#include <stdio.h> 
#include <stdlib.h> 
#include <stdint.h> 
#include <cs50.h> 

typedef uint8_t BYTE; 

/*struct jpg*/ 
/*{*/ 
/* BYTE soi[2] = { 0xFF, 0xD8 };*/ 
/* BYTE eoi[2] = { 0xFF, 0xD9 };*/ 
/*};*/ 

//SOI - 0xFF 0xD8 
//EOI - 0xFF 0xD9 
//APPn - 0xFF 0xEn  
int main(void) 
{ 
    //FAT - 512 bytes per block 
    BYTE block[512]; 

    //jpeg name 
    char name[6]; 

    bool is_open = false; 

    //JPEG 
    //struct jpg image; 

    //open file containing pictures 
    FILE* card_file = fopen("card.raw", "r"); 

    FILE* jpeg_file; 

    //make sure the file opened without errors 
    if (card_file == NULL) 
    { 
     printf("something went wrong and file could not be opened"); 
     return 1; 
    } 

    int i = 0; 
    while (fread(block, sizeof(BYTE), 512, card_file) != 0) 
    { 
     //jpeg start signature 
     if ((block[0] == 0xFF) && (block[1] == 0xD8) && (block[2] == 0xFF) && ((block[3] == 0xe1) || (block[3] == 0xe0))) 
     { 
      //assign jpeg name 
      sprintf(name, "%d.jpg", i++); 
      if(is_open) 
       fclose(jpeg_file); 

      //create a new jpeg file to copy bytes to 
      jpeg_file = fopen(name, "a+"); 
      is_open = true; 
     } 

     //write 512 bytes to a jpeg file 
     if(is_open) 
      fwrite(block, sizeof(block), 1, jpeg_file); 
    } 

    fclose(jpeg_file); 
    fclose(card_file); 
return 0; 
} 

Теперь не откажет, но только 9 из 50 JPEGs правильно восстановлены. cs50.h есть только так, что у меня есть доступ к типу bool. Каков лучший способ записать 50 файлов? У меня, похоже, есть логический недостаток с моими логическими выражениями.

Answer 1

fopen((char*)i, "w+"); полностью недействителен. Вы бросаете целое число как указатель, который будет сбой.

необходимо отформатировать число как имя файла:

char path[PATH_MAX]; 
sprintf(path, "%d", i); 
fopen(path, "w+"); 

Вы также не инициализируется jpeg_file - если условие не выполнено, jpeg_file будет дикий указатель, который также выходит из строя. Вы должны инициализировать jpeg_file по номеру NULL.

Answer 2

В своем fread вызовах вы должны передать адрес массива. Следовательно, утверждение должно быть fread(block, sizeof(BYTE), 512, card_file).

Постскриптум:

1. В вашем коде, есть предположение, что размер входного файла является целым кратным 512, который не нужен случай для JPEG файлов. Последние fread могут возвращать число меньше 512, которое должно быть обработано в вашей логике реализации. Следовательно, количество элементов для записи должно определяться возвращаемым значением fread

2. После завершения цикла вам необходимо закрыть указатель jpeg_file.

3. Последний, потому что вы работаете с JPEG, вы можете обрабатывать случай для EXIF ​​ файлы с иконками. В этом случае вы получите 2 SOI (начало изображения) маркеров.