ShuriZma
/
xenia-canary
mirror of https://github.com/xenia-canary/xenia-canary.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

further ajustment to buffer size to fix regression

This commit is contained in:
Cancerous 2019-12-06 12:20:34 -05:00 committed by illusion
parent c7270ad262
commit cbbc826fe4
1 changed files with 16 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
src/xenia/kernel/xam/xam_info.cc
View File

@ -361,22 +361,23 @@ dword_result_t XamEnumerate(dword_t handle, dword_t flags, lpvoid_t buffer,
}
}
// size_t actual_buffer_length = (uint32_t)buffer_length;
// if (buffer_length == e->items_per_enumerate()) {
// // Known culprits:
// // Final Fight: Double Impact (saves)
// XELOGW(
// "Broken usage of XamEnumerate! buffer length=%.X vs actual length=%.X "
// "(item size=%.X, items per enumerate=%u)",
// (uint32_t)buffer_length, actual_buffer_length, e->item_size(),
// e->items_per_enumerate());
size_t actual_buffer_length = (uint32_t)buffer_length;
if (buffer_length == e->items_per_enumerate()) {
// Known culprits:
// Final Fight: Double Impact (saves)
XELOGW(
"Broken usage of XamEnumerate! buffer length=%.X vs actual length=%.X "
"(item size=%.X, items per enumerate=%u)",
(uint32_t)buffer_length, actual_buffer_length, e->item_size(),
e->items_per_enumerate());
// actual_buffer_length = e->item_size() * e->items_per_enumerate();
// }
// leaving old method commented for now until more widely tested
// Don't trust buffer_length. It might be invalid (Based on "Resonance of fate")
// also fixes "Angry Birds" - Thanks Gliniak
size_t actual_buffer_length = e->item_count() * e->item_size();
actual_buffer_length = e->item_count() * e->item_size();
}
buffer.Zero(actual_buffer_length);