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

Str represented as read for local variable #22552

Closed
ShakeHakobyan opened this issue Jan 26, 2024 · 5 comments
Closed

Str represented as read for local variable #22552

ShakeHakobyan opened this issue Jan 26, 2024 · 5 comments
Assignees

Comments

@ShakeHakobyan
Copy link

Environment

Ուր 26 Հնվ 2024 07:22:08  +04
radare2 5.8.9 31576 @ linux-x86-64
birth: git.5.8.8-1011-g72e00141b3 2024-01-10__11:14:56
commit: 72e00141b32082394592a624ed8567fdb1051b3c
options: gpl -O? cs:5 cl:2 make
Linux x86_64

Description

I have mentioned this problem several times — some store-type instructions are marked as read for local variables. This is the result of "afv=" command.
image

Test

Here you can find my source file and binary file. Function name - "sym.foo3".

@ShakeHakobyan ShakeHakobyan changed the title Str represented as read from local variable Str represented as read for local variable Jan 26, 2024
@trufae
Copy link
Collaborator

trufae commented Jan 26, 2024

Can you provide the link to the other issues you reported this? Can you share a reproducer that i can use for testing? I assume and i know why this happens but without a repro its hard to say

@ShakeHakobyan
Copy link
Author

I have not reported this problem before. I have just seen it a few times in my files. I found another file in which I have the same problem. In the same folder , you can find that file and its source too. I am compiling my source file with the aarch64-linux-gnu-gcc command and then opening it with r2, in case it will be useful for you.

@ShakeHakobyan
Copy link
Author

In "edn" file the same problem is in the "sym.jpegdct" function.

@trufae
Copy link
Collaborator

trufae commented Mar 17, 2024

fix was just 1 line. pls confirm

@ShakeHakobyan
Copy link
Author

It's fixed, thank you!

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

2 participants