User talk:Leoetlino

From WiiBrew
Jump to navigation Jump to search

IOS syscall edits

Since you changed the function definitions for several of the syscalls, you want to fix their descriptions as well? Most of them no longer make sense because they refer to the old argument names. Tueidj (talk) 09:37, 1 May 2017 (CEST)

Oops, my bad. Sure, I've fixed the descriptions for the IOSC syscalls. leoetlino (talk) 13:16, 1 May 2017 (CEST)

ES content import/export key

I noticed you're fixing up ES stuff for Dolphin; FYI I'm pretty sure AddTMD and ExportTitleInit are meant to assign the PRNG key for content import/export (they're used mainly for copying DLC content between the SD card and NAND), not the title key from the ticket. For extra fun, there was a bug in the original IOS37 that caused the PRNG key data to be overwritten with zeroes when reading the korean key from the seeprom. They backed themselves into a corner with this bug (people had established large DLC collections on SD cards, all mistakenly encrypted with a NULL key) so when they fixed it in later IOS versions they added a check for the specific title IDs affected and use a zeroed key for those cases, and the PRNG key for all others. Tueidj (talk) 20:34, 20 June 2017 (CEST)

Oh wow, looks like I completely missed that... I wasn't sure whether it used the title key, but I was told it probably was, and I didn't bother to verify this -- my bad. You're absolutely right; I've had a look now, and I do see it uses key #5 if the active title is the system menu. Otherwise, it uses the all zero key (depending on the title type and ID). Thanks for letting me know about this. I don't have much time to fix this right now, but I will at some point :P leoetlino (talk) 10:18, 22 June 2017 (CEST)
Ugh, I keep finding new bugs and atrocities in the ES implementation. In particular, the CFD just keeps increasing to infinity :/ Looks like I'll need to do some more cleanup first before fixing ExportTitleInit/ImportTmd (because the export functions reuse the content table). leoetlino (talk) 15:18, 27 June 2017 (CEST)
Forgot to mention that this should be fixed by PR 5739. Of course, as it's IOS, there aren't many reviewers or reviews... so it may take a while before it makes its way into master. leoetlino (talk) 14:21, 20 July 2017 (CEST)