-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
FIX: exclude the latest capstone because it breaks ROPgadget #2492
base: stable
Are you sure you want to change the base?
Conversation
…Pgadget Signed-off-by: pmondon <[email protected]>
https://github.com/capstone-engine/capstone/blob/next/docs/cs_v6_release_guide.md#notes-about-aarch64-and-systemz-renaming
The coveralls CI breakage is unrelated. |
With the proposed fix there is no preventing the user from updating. |
We try to be as permissive as possible for package versions to make installing pwntools a no-brainer into any environment. The version restriction causes a conflict to be printed when trying out the new capstone and will result in the initial error in the ROP code too.
|
I understand the issue. Then I don't know how to fix this for now... |
This is only transitive though until ROPgadget is compatible with the capstone change. I've opened an issue for now JonathanSalwan/ROPgadget#202 |
I think adding that monkey patching before importing Line 601 in 34da249
It's weird that pip selects a pre-release version when resolving the dependencies. I would think this wasn't the case before. That version is out since 30.09. and this is the first time this pops up. |
What I did is downgrade capstone locally. |
Thanks for the fix, but I think pwntools is not the correct place to deal with this. |
The problem is with our dependency mentioning a prerelease version I'd be fine to let this sit for a while and wait for a fix from ropgadget. If you stumble on this while trying to use the ROP class with an aarch64 binary, there are workarounds posted here. |
FIX for this issue : #2491
The idea is that capstone push a new tag that is an Alpha and this breaks ROPgadget.
As ROPgadget does not have requirements to fix this issue (they only mention dependency on capstone) I thought that I could modify the requirements inside of pwntools so it excludes the breaking tag.