UDNL: Fix to GetFileStatFromImage

[PSI-Rockin]

Pull Request checklist

• I reformatted the code with clang-format
• I checked to make sure my submission worked
• I am the author of submission or have permission from the original author
• Requires update of the PS2SDK
• Requires update of the gsKit
• Others (please specify below)

Pull Request description

This PR fixes a minor bug in GetFileStatFromImage, where the UDNL clone would reject an image's IOPBTCONF if it had no ext info. However, the boot ROM UDNL will be successful regardless of the presence of any ext info.

[uyjulian]

cc @sp193

[sp193]

To avoid having multiple return paths (which was the original intent), would it be preferrable to move the goto end; (and the result = stat; assignment) to where you inserted that return statement?
Indeed, in the disassembly of UDNL, the file is accepted, even if there is no EXTINFO.

Alternatively, perhaps if you prefer so, we could replace the goto with two returns? Today, I would not use goto today, but this module was written many years ago.

[PSI-Rockin]

I have replaced the goto with two returns. There's a lot of cleanup that could be done for this, but I think that goes outside the scope of the PR.

[uyjulian]

Would you like me to fix the formatting, or would you like to do it yourself?

[PSI-Rockin]

Go ahead and fix the formatting.

[AKuHAK]

Would you like me to fix the formatting, or would you like to do it yourself?

BTW could you make CI format-check more verbose? As for now it just produces

Run make format-check
make format-check
shell: sh -e {0}
make: *** [Makefile:319: format-check] Error 1
##[error]Process completed with exit code 2.

As for me, I think that such type of command should be run completely from the shell, not from Makefile. It will make code maintaining more clear and portable.