Also, I think this is wrong.  Is there a reason you changed this?  I'm assuming you did not bother to test this change either.

https://github.com/hrydgard/ppsspp/blob/master/Core/Dialog/SavedataParam.cpp#L912

That function changes data.  My understanding was that it was supposed to do this *even if* it returns SCE_UTILITY_SAVEDATA_ERROR_RW_NO_MEMSTICK.  Did your tests show otherwise?  Or did you think the code just looked better this way so it ought to change, and no need to test it in any games?

-[Unknown]