fp = fopen(...);
if (!fp)
のほうがいい。