recover.c recovers all images successfully, but check50 says it fails to recover the last. Any ideas why this is happening?

Question

I have a while (!feof(infile)) loop, so that exits only when the eof is reached. And it closes all files upon exit. Here's the code:

// scan input file
while (!feof(infile))
{
    // read four bytes
    BYTE byte[4];
    fread(&byte, sizeof(BYTE), 4, infile);

Answer 1

feof is only set after fread tries to read the file and fails. When you reach the end of the file, fread will run one more time and your loop will carry on and write one more block (the last one, again). This will cause your last file to have an extra block in it, and fail check50.

Try to think of another way to know you have reached the end of the file.