Skip to content
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

ppc: fix registers overflow (next) #1688

Merged
merged 1 commit into from
Mar 19, 2021
Merged

ppc: fix registers overflow (next) #1688

merged 1 commit into from
Mar 19, 2021

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