Right — ferror() does not set errno, and so perror() is not appropriate here. fprintf(stderr, ...) would be better.