gh-142776: Ensure fp file descriptor is closed on all code paths in import.c

[stratakis]

• Issue: Leaked file descriptor in Python/import.c #142776

[brettcannon]

There's a potential uninitialized use of fp now.

[bedevere-app]

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

[stratakis]

There's a potential uninitialized use of fp now.

Oops. Refactored a bit, I believe it's better now.

[brettcannon]

Would you mind adding a news entry?

[bedevere-app]

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

[brettcannon]

@stratakis is this ready to go or did you have more that you wanted to do (you didn't leave the magic comment to signal you were done, although it appears you addressed everything).

[stratakis]

Yeah, everything done :)

Noting for the squash that I got a fixup commit there, you might not want to add the title of it to the commit message.