diff --git a/src/NDSCart.cpp b/src/NDSCart.cpp index 1f2fe692..a5fe3180 100644 --- a/src/NDSCart.cpp +++ b/src/NDSCart.cpp @@ -404,7 +404,11 @@ CartRetail::CartRetail(std::unique_ptr&& rom, u32 len, u32 chipid, bool ba { // Copy in what we can, truncate the rest. SRAM = std::make_unique(SRAMLength); memset(SRAM.get(), 0xFF, SRAMLength); - memcpy(SRAM.get(), sram.get(), std::min(sramlen, SRAMLength)); + + if (sram) + { // If we have anything to copy, that is. + memcpy(SRAM.get(), sram.get(), std::min(sramlen, SRAMLength)); + } } } @@ -1650,7 +1654,8 @@ std::unique_ptr ParseROM(std::unique_ptr&& romdata, u32 romlen } std::unique_ptr cart; - auto [sram, sramlen] = args ? std::move(*args->SRAM) : std::make_pair(nullptr, 0); + std::unique_ptr sram = args ? std::move(args->SRAM) : nullptr; + u32 sramlen = args ? args->SRAMLength : 0; if (homebrew) cart = std::make_unique(std::move(cartrom), cartromsize, cartid, romparams, args ? std::move(args->SDCard) : std::nullopt); else if (gametitle[0] == 0 && !strncmp("SD/TF-NDS", gametitle + 1, 9) && gamecode == 0x414D5341) diff --git a/src/NDSCart.h b/src/NDSCart.h index 7d482ab0..78439a2c 100644 --- a/src/NDSCart.h +++ b/src/NDSCart.h @@ -62,9 +62,14 @@ struct NDSCartArgs std::optional SDCard = std::nullopt; /// Save RAM to load into the cartridge. - /// If \c nullopt, then the cart's SRAM buffer will be empty. + /// If \c nullptr, then the cart's SRAM buffer will be empty. /// Ignored for homebrew ROMs. - std::optional, u32>> SRAM = std::nullopt; + std::unique_ptr SRAM = nullptr; + + /// The length of the buffer in SRAM. + /// If 0, then the cart's SRAM buffer will be empty. + /// Ignored for homebrew ROMs. + u32 SRAMLength = 0; }; // CartCommon -- base code shared by all cart types diff --git a/src/frontend/qt_sdl/ROMManager.cpp b/src/frontend/qt_sdl/ROMManager.cpp index b065ad1a..a20af206 100644 --- a/src/frontend/qt_sdl/ROMManager.cpp +++ b/src/frontend/qt_sdl/ROMManager.cpp @@ -1316,8 +1316,8 @@ bool LoadROM(EmuThread* emuthread, QStringList filepath, bool reset) // the ROM is homebrew or not. // So this is the card we *would* load if the ROM were homebrew. .SDCard = GetDLDISDCardArgs(), - - .SRAM = std::make_pair(std::move(savedata), savelen), + .SRAM = std::move(savedata), + .SRAMLength = savelen, }; auto cart = NDSCart::ParseROM(std::move(filedata), filelen, std::move(cartargs));