Skip to content

Commit ed04a86

Browse files
committed
[DynaRec Disassembler] Changed disassemblerBuffer() to return 0 value for bufferSize when errors occur
1 parent 68dc687 commit ed04a86

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/gui/widgets/dynarec_disassembly.cc

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ void PCSX::Widgets::Disassembly::writeFile() {
3131
std::ofstream file;
3232
// Open file - default location in resources directory
3333
file.open("DynarecDisassembly.txt", std::ios::app);
34-
// If file exists, write to it, otherwise return -1
34+
// If file exists, write to it, otherwise return
3535
if (file) {
3636
for (auto i = 0; i < m_items.size(); ++i) {
3737
file << m_items[i];
@@ -163,17 +163,17 @@ size_t PCSX::Widgets::Disassembly::disassembleBuffer() {
163163
if (buffer == nullptr) {
164164
PCSX::g_system->printf("Disassembler Error: nullpointer to code buffer.\n");
165165
m_showError = true;
166-
return -1;
166+
return 0;
167167
} else if (bufferSize <= 0) {
168168
PCSX::g_system->printf("Disassembler Error: Invalid code buffer size.\n");
169169
m_showError = true;
170-
return -1;
170+
return 0;
171171
}
172172
// Attempt to initialize Capstone disassembler, if error log it and return
173173
if (cs_open(CS_ARCH, CS_MODE, &handle) != CS_ERR_OK) {
174174
PCSX::g_system->printf("Disassembler Error: Failed to initialize Capstone.\n");
175175
m_showError = true;
176-
return -1;
176+
return 0;
177177
}
178178
// Set SKIPDATA option as to not break disassembler
179179
cs_option(handle, CS_OPT_SKIPDATA, CS_OPT_ON);
@@ -196,7 +196,7 @@ size_t PCSX::Widgets::Disassembly::disassembleBuffer() {
196196
cs_close(&handle);
197197
PCSX::g_system->printf("Disassembler Error: Failed to disassemble buffer.\n");
198198
m_showError = true;
199-
return -1;
199+
return 0;
200200
}
201201
// Successful disassembly, clean up disassembler instance and return successful result
202202
cs_close(&handle);

0 commit comments

Comments
 (0)