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

test_riscv.py failing #2405

Closed
KelvinChung2000 opened this issue Jul 23, 2024 · 3 comments · Fixed by #2456
Closed

test_riscv.py failing #2405

KelvinChung2000 opened this issue Jul 23, 2024 · 3 comments · Fixed by #2456
Labels
bug RISCV Arch Testing Test related issue
Milestone

Comments

@KelvinChung2000
Copy link

Work environment

Questions Answers
OS/arch/bits Pop os
Architecture x86
Source of Capstone pip
Version/git commit v5.0.1

Expected behavior

When running the bindings/python/tests/test_riscv.py should not be fault.

Actual behavior

Getting the following:
File "/home/kelvin/capstone/./bindings/python/tests/test_riscv.py", line 45, in print_insn_detail
if i.access == CS_AC_READ:
AttributeError: 'RISCVOp' object has no attribute 'access'

Steps to reproduce the behavior

calling python3 ./bindings/python/tests/test_riscv.py from the root directory of Capstone git clone

Additional Logs, screenshots, source code, configuration dump, ...

image

I found this problem because when I try to use regs_access for a CsInsn, which I am also getting errors. This led me to find an example and find this. The cause of the error is not the same. Should I open another issue about that?

@Rot127 Rot127 added bug RISCV Arch Testing Test related issue labels Jul 24, 2024
@Rot127 Rot127 added this to the v6 milestone Jul 24, 2024
@Rot127
Copy link
Collaborator

Rot127 commented Jul 24, 2024

I currently rewrite testing. So this one will be fixed with #2384

@Rot127 Rot127 mentioned this issue Jul 24, 2024
5 tasks
@KelvinChung2000
Copy link
Author

I would also like to ask about the regs_access issue. Should I open another issue for a bug report?

@Rot127
Copy link
Collaborator

Rot127 commented Jul 24, 2024

Ah yes, please do so.

@Rot127 Rot127 mentioned this issue Aug 25, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug RISCV Arch Testing Test related issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants