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

Find path between functions does not work #23697

Open
trufae opened this issue Nov 27, 2024 · 3 comments
Open

Find path between functions does not work #23697

trufae opened this issue Nov 27, 2024 · 3 comments
Assignees
Milestone

Comments

@trufae
Copy link
Collaborator

trufae commented Nov 27, 2024

Seems like the xrefs rewrite broke it. we lack tests for this so it was expected i guess.

reproducer:

r2 -a arm test/bins/mach0/ls-m1

[0x100005240]> axt $FB
sym.func.1000047a8 0x100004814 [CALL:--x] bl sym.func.100004b34
sym.func.1000047a8 0x1000049c0 [CALL:--x] bl sym.func.100004b34
[0x100005240]> s sym.func.1000047a8
[0x1000047a8]> axt $FB
main 0x100004614 [CALL:--x] bl sym.func.1000047a8
[0x1000047a8]> s main
[0x100003a90]> axt $FB
[0x100003a90]> abp?
| abp[?] [addr]  follow basic blocks paths from $$ to `addr`
[0x100003a90]> abp 0x100005240
[0x100003a90]> abp 0x100005240 @ entry0
[0x100003a90]> abl,size/sort/inc|tail -n 5
0x1000041bc 124  1      31     0x10000424c 0x100004238  0x100003a90  0x1000078c0 0x1000078c0 0x1000078c0 0x1000078c0 0x1000078c0              0x1000078c0 0x1000078c0 0x1000078c0 0x1000078c0 0x1000078c0
0x100005b54 124  1      31     0x100005bd0              0x100005638  0x1000074a0 0x1000074c0 0x100007750 0x1000075d0                          0x1000074a0 0x1000074c0 0x100007750 0x1000075d0
0x1000042fc 132  1      33     0x1000043c4 0x100004380  0x100003a90
0x100005240 140  1      35     0x1000052d4 0x1000052cc  0x100004b34  0x1000077e0 0x1000077e0 0x1000077e0                                      0x1000077e0 0x1000077e0 0x1000077e0

[0x100003a90]> o
 3 * r-x 0x0002daa0 ls-m1
 4 - r-- 0x00002560 null://9568
 5 - rw- 0x000f0000 malloc://983040
[0x100003a90]>
@trufae trufae added this to the 6.0.0 milestone Nov 27, 2024
@trufae
Copy link
Collaborator Author

trufae commented Dec 1, 2024

i think that issue is related to the refactoring of the xrefs storage using RVec. and actually it's probably the same issue spotted in #23096 in case you are in the mood to check another similar issue :)

@condret
Copy link
Member

condret commented Dec 6, 2024

can we close this now?

@trufae
Copy link
Collaborator Author

trufae commented Dec 7, 2024

Lets reword it to implement proper algo for abpf like it is for abp

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

No branches or pull requests

3 participants