Well, I agree.  This fixes the crash but probably the code doing that should be changed as well.  It's actually the other code I modified as well:

```
u8* tmpData = sfoFile.GetValueData("SAVEDATA_FILE_LIST", 0);
<allocate tmpData if != 0>
...
<change to tmpData>
...
sfoFile.SetValue("SAVEDATA_FILE_LIST", tmpData, 3168, 3168);
<deallocate iff allocated above>
```

Probably cleaner to just always make a copy.  Should I change that too and remove this part of the change?

-[Unknown]