Closed phil-el closed 6 months ago
There is a missing break, after this line: https://github.com/nemerle/dcc/blob/f9b89c052b3e8dea85745697ad91e41c0dde572e/3rd_party/libdisasm/ia32_modrm.cpp#L156 The bug is also in the original libdisasm since at least 15 year. It was probably not noticed because [bx+di] addressing mode was rarely used by compiler., none of the .exe test use it.
No patch provided as this repo seems inactive and anyway the patch is trivial but people who want to fork it need to care about that.
There is a missing break, after this line: https://github.com/nemerle/dcc/blob/f9b89c052b3e8dea85745697ad91e41c0dde572e/3rd_party/libdisasm/ia32_modrm.cpp#L156 The bug is also in the original libdisasm since at least 15 year. It was probably not noticed because [bx+di] addressing mode was rarely used by compiler., none of the .exe test use it.
No patch provided as this repo seems inactive and anyway the patch is trivial but people who want to fork it need to care about that.