Question

代码必须从具有原始JPEG的文件一次读取512个字节才能检索它们。当找到JPEG时，所有以下512字节的块都存储在一个文件中，直到找到新的JPEG，一旦找到新的JPEG，文件img就会关闭然后再打开以存储新的图像。如果我评论或删除fclose（img），代码运行正常;但是，它表示上面的错误。

这是代码

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


int main (int argc, char *argv[]){

if(argc != 2){

    fprintf(stderr, "Usage: ./recovery image\n");
    return 1;

}

FILE *card = fopen(argv[1], "r");

if (card == NULL){

    fprintf(stderr, "Cannot open image\n");
    return 2;

}

FILE *buffer1 = malloc(sizeof(card));

int n = 0;
int picsNum = 0;

while(fread(buffer1, 512, 1, card)){

    n ++;

}

//printf("%i", n);

bool foundJPEG = false;
int j = 0;
bool fstJPEG = false;
FILE *img = NULL;
int counter = 0;

fseek(card, 0, SEEK_SET);

for(int i = 0; i < n; i++){


    unsigned char fst;
    unsigned char snd;
    unsigned char thrd;
    unsigned char fth;
    unsigned char buffer2[508];

    fread(&fst, 1, 1, card);
    fread(&snd, 1, 1, card);
    fread(&thrd, 1, 1, card);
    fread(&fth, 1, 1, card);
    fread(buffer2, 508, 1, card);
    //printf("%i,%i,%i,%i,", fst, snd, thrd, fth);

    if(fst == 0xff && snd == 0xd8 && thrd == 0xff && (fth & 0xf0) == 0xe0){

        printf("Found\n");

        if(fstJPEG){

            fclose(img);

        }

        printf("%i,%i,%i,%i,", fst, snd, thrd, fth);
        foundJPEG = true;
        picsNum++;
        fstJPEG = true;

        char JPEG[8];
        sprintf(JPEG, "%03i.jpg", j);
        j++;
        img = fopen(JPEG, "w");

    } else {

        foundJPEG = false;

    }

    if(fstJPEG){

        fwrite(&fst, 1, 1, card);
        fwrite(&snd, 1, 1, card);
        fwrite(&thrd, 1, 1, card);
        fwrite(&fth, 1, 1, card);
        fwrite(buffer2, 508, 1, card);
        counter++;
        printf("%i,", counter);

    }

}    

fclose(card);    
printf("%i,\n", picsNum);

}

Answer 1

这在多方面都是错误的：

FILE *buffer1 = malloc(sizeof(card));

首先，卡片是FILE *，所以这会分配一个指针，而不是FILE。其次，您无法以这种方式打开文件。您必须使用fopen()或类似的东西。

Answer 2

您的想法

FILE *buffer1 = malloc(sizeof(card));

正在做什么？它实际上是创建一个4或8字节长的缓冲区。（文件大小*）。你被告知一次读取512个字节。它的char *缓冲区不是文件*。这样做。

char *buffer1 = malloc(512);

解决`./recover'中的错误：free（）：无效的下一个大小（正常）：0x0000000001ddd270 ***在c中止

2 个答案: