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

Process: Correct AUXV to support 64bit Linux #178

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

arilou
Copy link
Contributor

@arilou arilou commented Jul 5, 2022

Signed-off-by: Jon Doron jond@wiz.io

Signed-off-by: Jon Doron <jond@wiz.io>
@eminence
Copy link
Owner

eminence commented Jul 5, 2022

Thanks. I'm embarrassed to say that function has been broken ever since I added it 4 years ago (307aa87)!

@eminence eminence merged commit 5468a25 into eminence:master Jul 5, 2022
@arilou
Copy link
Contributor Author

arilou commented Jul 6, 2022

Heh no problem, thanks for merging so quickly :)
I was hoping if you are willing to expose /proc/X/mem file out, so we can read/write to a process virtual memory
(now that AUXV is valid it makes more sense)

What do you think?

@eminence
Copy link
Owner

This fix has been released in procfs v0.13.1. Note that this change is technically an API breaking change, but since this function has been broken for 4 years, I feel confident that no one was actually using it (except you 😄 ) Thanks again for the fix

@arilou
Copy link
Contributor Author

arilou commented Jul 11, 2022

That's awesome 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

Successfully merging this pull request may close these issues.

2 participants