-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DiscIO: Improve error handling for reading integers #3127
Conversation
|
||
bool CBlobBigEndianReader::Read16(u64 offset, u16* buffer) const | ||
{ | ||
if (!m_reader.Read(offset, sizeof(u16), (u8*)buffer)) |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
94423eb
to
21c8743
Compare
Fixes the crash when loading game list. |
4c5db4b
to
4ceb33a
Compare
e8fd65d
to
f131d94
Compare
Any reason not to merge this? |
|
||
bool Read32(u64 offset, u32* buffer) const; | ||
bool Read16(u64 offset, u16* buffer) const; | ||
bool Read8(u64 offset, u8* buffer) const; |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
LGTM |
b61f46b
to
91b78e9
Compare
Bump |
Callers can now check whether reads fail, either by checking the return value or by setting the buffer to a known bad value and seeing if it stays untouched. I've added error checks to FileSystemGCWii and Boot_BS2Emu, but not to Boot since it doesn't check any of its other reads either.
91b78e9
to
cbfab2a
Compare
FifoCI detected that this change impacts graphical rendering. Here are the behavior differences detected by the system:
automated-fifoci-reporter |
DiscIO: Improve error handling for reading integers
For https://bugs.dolphin-emu.org/issues/9002