FlagBrew / PKSM

Gen I to GenVIII save manager.
GNU General Public License v3.0
1.67k stars 173 forks source link

Bug Report - Exception occurs when moving box #1464

Open Badhaas opened 4 days ago

Badhaas commented 4 days ago

Describe the bug Various crashes, exceptions and other weird behaviour occur when moving a box of Pokémon from an OR save file into an AS (or X or Y) save file. More details about repro and various cases below.

Variations:

To Reproduce Steps to reproduce the behaviour:

  1. Move a full box of Pokémon from Omega Ruby into PKSM storage (with the green indicator, selecting the entire box)
  2. Save, exit, restart with a different cartridge
  3. Load a save from a different game:
    • European cartridge, language set to Japanese
  4. Try to move the full box from PKSM storage into the game (with the green indicator, selecting the entire box)
  5. Notice PKSM crash (or if it doesn't, it changes
    • Or if it doesn't, notice weird ? boxes replacing characters in some Pokémon's OTs

Transfers in which this happened:

Expected behaviour Pokémon can be moved and cloned without crashing or corrupting Pokémon data.

Screenshots

PXL_20240701_074107591 PXL_20240701_074422283 PXL_20240701_080401823

Information about your environment

Additional context I've got a bunch of crash dumps, let me know if you need those as well.

PKSM Crash / error scenarios + when it doesn't happen:

EDIT: Modified the repro as I noticed at happens for pretty much all boxes. Left in the additional context notes, but this seems to be a much more common issue in general moving boxes with anything ORAS involved.

Badhaas commented 3 days ago

After further investigation, I've ruled out it's caused by corrupt Pokémon data.

To check whether it makes sense:

Since PkHex can load all Pkm, and "verify all Pkm" indicates there's no problems, I don't think this is caused by corrupt Pkm data.