#8 Fix region parsing

Geschlossen
Ghost möchte 0 Commits von deleted nach necklace/master zusammenführen
Ghost kommentierte vor 3 Jahren

As listed on PS3DevWiki, the int32 at the start of the first sector is actually the number of unencrypted regions, not the total regions divided by 2. Also, the region ends are inclusive (e.g. the sector region 0-5151 is actually the byte range 0-10551296 rather than 0-10549248).

This PR changes the read_regions function to fix both of those issues, as they led to the last sector of each unencrypted region being encrypted/decrypted when it shouldn't have been.

[As listed on PS3DevWiki](https://www.psdevwiki.com/ps3/Bluray_disc#Encryption), the int32 at the start of the first sector is actually the number of unencrypted regions, not the total regions divided by 2. Also, the region ends are inclusive (e.g. the sector region 0-5151 is actually the byte range 0-10551296 rather than 0-10549248). This PR changes the read_regions function to fix both of those issues, as they led to the last sector of each unencrypted region being encrypted/decrypted when it shouldn't have been.
Nichlas Severinsen kommentierte vor 3 Jahren
Besitzer

It is imperative that all code contributed be legitimately free software. For that reason, code from anonymous (or pseudonymous) contributors will not be accepted.

Personally I don't see the point in sending anonymous PRs. Why bother fixing the code if you're not going to take credit for it? Next time just make issues. Thanks.

It is imperative that all code contributed be legitimately free software. For that reason, code from anonymous (or pseudonymous) contributors will not be accepted. Personally I don't see the point in sending anonymous PRs. Why bother fixing the code if you're not going to take credit for it? Next time just make issues. Thanks.
necklace hat vor 3 Jahren geschlossen
Bitte diesen Pull-Request wieder eröffnen, um die Merge-Operation auszuführen.
Anmelden, um an der Diskussion teilzunehmen.
Kein Meilenstein
Niemand zuständig
2 Beteiligte
Laden…
Abbrechen
Speichern
Hier gibt es bis jetzt noch keinen Inhalt.