BernardoGiordano / Checkpoint

Fast and simple homebrew save manager for 3DS and Switch.
GNU General Public License v3.0
2.55k stars 150 forks source link

Attempt to restore a modified Minecraft save on Switch resulted in error & reboot #114

Closed Willgheminass closed 6 years ago

Willgheminass commented 6 years ago

Issue template

Please fill the fields with all the informations important for the maintainers to get your issue fixed or sorted out.

If you feel it's relevant, please attach screenshots or pictures that could detail the issue more.

Error details

PuppyTails commented 6 years ago

I've Noticed this happens when modifying (modify-ing?) a save on Mac, but the error didn't happen when I modified the save on windows, I could be wrong.

Edit: Deleting the save on the Switch then restoring seems to fix. Edit 2: Fixed spelling issue.

BernardoGiordano commented 6 years ago

Are you using the modified hekate files for layeredfs?

IntaroArt commented 6 years ago

I got this error as well, some kind of issue with being in airplane mode.

1) disconnect from wifi via clear wifi settings 2) turn off airplane mode

Edit: scratch that, worked a few times and then it gives the same error

BernardoGiordano commented 6 years ago

The solution is to use stable Hekate files.

CarJem commented 6 years ago

Like he said, the solution is to use stable Hekate files, with LayerFS. Here's a link for everyone who is lost: https://github.com/tumGER/SDFilesSwitch/releases/tag/v4

Please Close this issue though.

EDIT: Maybe not, the issue is very much a problem, I have wiped, then attempted to restore, many crashes, getting nowhere. Maybe all we really need is an actual save editor for bedrock. EDIT 2: Need to find the actual Hekate Files

BernardoGiordano commented 6 years ago

I meant official Hekate Files without the shitty preliminar and unstable LayeredFS support.

CarJem commented 6 years ago

Sorry, where do I get those again, still relatively new to homebrew?

EDIT: Ignore I have a follow-up post.

CarJem commented 6 years ago

I have no idea what is up with this issue, but I've heard plenty of people who have no issues with importing and restoring more than once, and I am unsure if this has something to do with the Firmware Version, CFW or LayerFS (Which I don't recommend personally recently), or Checkpoint itself. Whatever the case is this should be investigated and not ignored. I am having plenty of headaches with this problem, and often waste hours upon hours trying to restore with a modified save.

Some of the things I have experienced:

Some of the things I've heard about:

I am super confused on who's fault this issue may be caused by, whether it be LayerFS/CFW, the Firmware being used, or Checkpoint itself. I am also unable to find the "Official" files that have been mentioned by @BernardoGiordano , so some clarification there would be appreciated for those who are confused.

EDIT: Perhaps this could have something to do with File Permissions, which vary between OS's.

SMNFXCN commented 6 years ago

I have this problem too. It happens when you try to restore/backup a save after modifying once.

fennectech commented 6 years ago

Ive solved this problem by deleting the minecraft savedata from horizon (THEN OPENING THE GAME THIS IS IMPORTANT) then restoring the save. (You must close the game before the Mojang logo appears but after the game leaves the home screen. Close it while it shows the nintendo switch logo screen.

fennectech commented 6 years ago

When the issue crops up you cannot restore any save to bedrock Not even vanilla untouched saves.

fennectech commented 6 years ago

Sometimes these problematic saves also cause nxshell to crash when interacting with them. Nxshell seems to be having issues with path length. Perhaps that’s effecting here.

Woody601 commented 6 years ago

@fennectech are you 100% sure that method you posted above works? I am iffy on deleting the savefile then trying to restore it and then fail like it does when restoring as I don't want to lose all of my hard work...

fennectech commented 6 years ago

Its the only way ive gotten it to work. If you know your save is good thats thway i do it.

BernardoGiordano commented 6 years ago

Can you test with this build? https://www.dropbox.com/s/63cuarl6jou2eh9/Checkpoint.nro?dl=0

IntaroArt commented 6 years ago

tested it out. Was able to "restore" a modified save file once. Tried restoring to a non-modified state and a different modified game save and received error 0000F601 and the same crash error code( 2011-0301 ).

fennectech commented 6 years ago

Same thing for me on that buid.

BernardoGiordano commented 6 years ago

And with this one? https://www.dropbox.com/s/33c2gk32crfjap3/Checkpoint.nro?dl=0

IntaroArt commented 6 years ago

Tried it out again with the newer one you linked. Completely deleting my save game data and launching the game again so it would be a clean save file. Was able to restore once correctly. After checking and attempting again I same error. save error 0000F601 - Could not mount save state. But I couldn't see a crash code because the console froze and wouldn't show any crash error. After pressing the power button it would reboot without showing a crash error.

BernardoGiordano commented 6 years ago

Can you send me both original and modified Minecraft saves you're using?

IntaroArt commented 6 years ago

https://www.dropbox.com/s/5l19h3xvtoufjh8/MC-Checkpoint.zip?dl=0 These are the two save files used. Clean being the original save, the other adding one map. I've been using Tumger's SD files and launching with CFW. I've tried with layeredFS and it has more issues (like you said but I was using it for a game mod). The same error occurs in EdiZon as well.

877cms commented 6 years ago

Same issue here with BOTW, using RajNX CFW and lastest Checkpoint 3.4.1.

Backs saves up ok on all other games. On BOTW it backs up ok, but trying to restore gets me: Error 0000F601 Failed to commit save to device

UPDATE - NOW WORKING FINE, BUT HAD TO USE HEKATE 3.2

IntaroArt commented 6 years ago

@877cms I've had that issue as well (never used RajNX) I believe RajNX has LayeredFS enabled by default which caused that error instantly. I think it is just an issue with layeredFS and checkpoint because it should work mostly fine with CFW launched by hekate but like Bernardo said it will work best with the official hekate files.

877cms commented 6 years ago

@IntaroArt yes worked fine with offical Hekate 3.2, but problems with rajnx and of course sxos didn't work either. Not sure how rajnx differs from original hekate but there is obviously something. PS you can select layerfs or not with rajnx, I was choosing not.

UPDATE: I'm now updated to 5.1.0 and Checkpoint 3.4.1 is working fine with SXOS 1.5

BernardoGiordano commented 6 years ago

Does this work? https://www.dropbox.com/s/wz9ca7pb27hc8l1/Checkpoint.nro?dl=0

fennectech commented 6 years ago

Failed to commit to device 0000F601 on hekate 4.0 with latest layeredfs and sdfiles.

BernardoGiordano commented 6 years ago

@fennectech can you test it without layeredfs?

IntaroArt commented 6 years ago

@BernardoGiordano Tested on 5.1.0, Hekate 4.0, CFW (no Layered FS) tomGer's SDfiles 8, it works the first time I try to restore a backup. Any time after that error 0000F601 and/or Crash 2011-0301. The only way I can get around this is completely deleting the save data on the switch, launching the game, go to checkpoint to restore a modified/original save. It is not the most convenient solution but its something I personally can live with.

vigigure commented 6 years ago

jksm on switch can backup and restor miencraft saves for me https://github.com/J-D-K/JKSV

fennectech commented 6 years ago

It also can for me. Though its far less optimised and i perfer this one.

vigigure commented 6 years ago

why is it not optimised? it doesn't slow for me

fennectech commented 6 years ago

Its not that its slow The menu is rather ugly and interface feels clunky. _

vigigure commented 6 years ago

it is just a grid of icns with a box to confirm stuff and use the same colors as switch. i rather use something that actually works instead of delete save data over and over to make it work

BernardoGiordano commented 6 years ago

@vigigure we're debugging this to make it work fyi

BernardoGiordano commented 6 years ago

@vigigure @fennectech @877cms @Woody601 @TheProvider0 @PuppyTails @IntaroArt @CarJem @98otiss03

Can you all test with this version? https://www.dropbox.com/s/wz9ca7pb27hc8l1/Checkpoint.nro?dl=0

Also, if this does work as intended, please try to see if it works with other cfw that also enable modules like layeredfs and fs_mitm

IntaroArt commented 6 years ago

@BernardoGiordano Tested (same configuration as last time but with the "newer" test version of checkpoint. Was able to successfully restore twice without deleting data! Tested with "Newest LayeredFS" via tomger's SD files: no issues, was able to restore the save files without any errors or crashes!

It seems like this version is working Only issues are using older save data files after a newer update (not checkpoint's issue as far as I can tell)

fennectech commented 6 years ago

Works for me. This bug was a tricky bastard!

Can you guys add a button on the keyboard input screen to accept the current text. (Basically a button equivalent of pressing okay. Like start or y or something like that. So that we don’t have to go to “okay” to use the default filename (Edit i implimented it myself and submitted a PR)

@BernardoGiordano Can you push this fix somewhere so i can compile a copy on my own? (would like to add my keyboard change into it

fennectech commented 6 years ago

NEVERMIND I GOOFED Got confused by Checkpoint(1).nro Checkpoint(2).nro Checkpoint(27).nro etc xD

BernardoGiordano commented 6 years ago

This is now considered fixed.