Skip to content

Conversation

@StalkR
Copy link
Contributor

@StalkR StalkR commented Sep 15, 2020

@StalkR StalkR changed the title ppc: fix registers overflow ppc: fix registers overflow (next) Sep 15, 2020
@catenacyber
Copy link
Contributor

@StalkR for some context information, this bug comes from llvm cf https://reviews.llvm.org/D61935

The good way to fix it would be to sync with latest LLVM cf #1510

@StalkR
Copy link
Contributor Author

StalkR commented Sep 16, 2020

OK good to know, up to @aquynh then.

When will #1510 be merged?

If not soon, this is a small patch just for the overflow the fuzzers are hitting (Regs[RegNo] with RegNo beyond Regs) that could be used in the meantime.

@catenacyber
Copy link
Contributor

If not soon, this is a small patch just for the overflow the fuzzers are hitting (Regs[RegNo] with RegNo beyond Regs) that could be used in the meantime.

Sounds good.

@aquynh I guess you will not update PPC soon, so this workaround seems fair.

@StalkR
Copy link
Contributor Author

StalkR commented Jan 18, 2021

friendly ping @aquynh

(I just sync'd and rebased the pull request since it's been a while)

@pranith
Copy link
Contributor

pranith commented Mar 4, 2021

Can you please create a new PR on libcapstone?

@aquynh aquynh merged commit 702dbe7 into capstone-engine:next Mar 19, 2021
@aquynh
Copy link
Collaborator

aquynh commented Mar 19, 2021

merged, thanks!

@StalkR StalkR deleted the next-ppcregsoverflow branch January 2, 2022 00:17
@riptl riptl mentioned this pull request Jul 22, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants