Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VolumeVerifier: Use correct IOS filename for development discs. #10028

Merged
merged 1 commit into from Aug 16, 2021

Conversation

GerbilSoft
Copy link
Contributor

Retail-signed discs use the format: IOS56-64-v5661.wad
Debug-signed discs use the format: firmware.64.56.22.29.wad

Debug-signed discs usually have a 128 version of the firmware as well, since some devkits have 128 MB MEM2. (Retail has 64 MB.)

This fixes part of VolumeVerifier for debug discs. It's not validating the signatures properly. I attempted to add the debug root key (which is different from retail), and that helped, but ESDevice::VerifyContainer() was still getting an error from mbedtls, even though the public key appeared to be correctl

@GerbilSoft GerbilSoft force-pushed the bugfix/RVT-firmware-filenames branch from ed93a76 to f026405 Compare August 15, 2021 02:25
@JosJuice
Copy link
Member

I don't have any development discs to verify this with, but I'll trust you that this is correct. However you'll need to fix the lint error before this can be merged.

Retail-signed discs use the format: IOS56-64-v5661.wad
Debug-signed discs use the format:  firmware.64.56.22.29.wad

Debug-signed discs usually have a 128 version of the firmware as well,
since some devkits have 128 MB MEM2. (Retail has 64 MB.)
@GerbilSoft GerbilSoft force-pushed the bugfix/RVT-firmware-filenames branch from f026405 to 95d3416 Compare August 15, 2021 16:41
@GerbilSoft
Copy link
Contributor Author

Updated to switch to u64 and fix the lint error.

I have several RVT-R images, all of which have WADs in the update partition similar to the following:

firmware.128.56.21.29.wad
firmware.64.56.21.29.wad

These are both the same IOS version (equivalent to retail IOS56 v5405). The 64 version is for systems with 64 MB MEM2 (RVT-R Reader), while the 128 version is for systems with 128 MB MEM2 (RVT-H Reader, NDEV).

@Tilka Tilka merged commit 0f628ff into dolphin-emu:master Aug 16, 2021
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants