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 bd025b1f4f
commit 4b71327673
1 changed files with 16 additions and 15 deletions

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());
// 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
actual_buffer_length = e->item_count() * e->item_size();
}
// 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();
buffer.Zero(actual_buffer_length);