Skip to content

Replace CS_ARCH_ARM64 with CS_ARCH_AARCH64 #208

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

scriptk1d
Copy link

capstone not contain CS_ARCH_ARM64

@scriptk1d
Copy link
Author

File ~/.local/lib/python3.12/site-packages/ropgadget/core.py:70, in Core.__getGadgets(self)
     67 if not self.__checksBeforeManipulations():
     68     return False
---> 70 G = Gadgets(self.__binary, self.__options, self.__offset)
     71 execSections = self.__binary.getExecSections()
     73 # Find ROP/JOP/SYS gadgets

File ~/.local/lib/python3.12/site-packages/ropgadget/gadgets.py:24, in Gadgets.__init__(self, binary, options, offset)
     22 if self.__arch == CS_ARCH_X86:
     23     re_str = "db|int3"
---> 24 elif self.__arch == CS_ARCH_ARM64:
     25     re_str = "brk|smc|hvc"
     26 if self.__options.filter:

NameError: name 'CS_ARCH_ARM64' is not defined

@SweetVishnya
Copy link
Collaborator

I suppose, this is a duplicate of #202

Is stable capstone release out there?

@SweetVishnya
Copy link
Collaborator

I suppose we should wait for a stable capstone 6 release, and then also update the requirements

Copy link
Collaborator

@SweetVishnya SweetVishnya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Waiting for a stable capstone 6 release

Also, we should change the minimum required capstone version to 6

@scriptk1d
Copy link
Author

Thank you for the clarification! I understand that this issue is indeed a duplicate of #202. Let's look forward to the stable release of Capstone 6 and hope it addresses the related bugs.

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.

2 participants