Fix incorrect OP_CLOSURE decompilation (register C -> Bx)#73
Open
jalbert-dev wants to merge 1 commit intoviruscamp:masterfrom
Open
Fix incorrect OP_CLOSURE decompilation (register C -> Bx)#73jalbert-dev wants to merge 1 commit intoviruscamp:masterfrom
jalbert-dev wants to merge 1 commit intoviruscamp:masterfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix #74.
The decompiler currently handles
OP_CLOSUREimproperly, and looks for the function index in register C, when Lua uses combined register Bx. This will actually work as expected with vanilla Lua implementations that use the typicalOp-A-C-Bopcode-operand ordering, as C happens to be the lower 9 bits of Bx, but will fail with Lua implementations that use a different ordering.This patch makes some tiny changes to
ProcessCodeso that it reads the function index from Bx instead.(The disassembler part of luadec already reads from Bx instead of C, so no changes are necessary there.)