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

fix kallsyms in libbpfgo helpers #153

Merged
merged 3 commits into from Jun 27, 2022

Conversation

itamarmaouda101
Copy link
Contributor

@itamarmaouda101 itamarmaouda101 commented Apr 19, 2022

kallsyms helper is including [] chars when it shouldn't.

This PR solves it by using TrimPrefix and TrimSurfix.

@grantseltzer grantseltzer self-requested a review April 21, 2022 17:03
@rafaeldtinoco rafaeldtinoco marked this pull request as ready for review June 24, 2022 04:39
@rafaeldtinoco rafaeldtinoco requested review from grantseltzer and removed request for grantseltzer June 24, 2022 04:39
@rafaeldtinoco rafaeldtinoco added this to the v0.3.0-libbpf-0.8.0 milestone Jun 27, 2022
Signed-off-by: grantseltzer <grantseltzer@gmail.com>
Signed-off-by: grantseltzer <grantseltzer@gmail.com>
Copy link
Contributor

@grantseltzer grantseltzer left a comment

Choose a reason for hiding this comment

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

L G T M !

@grantseltzer grantseltzer merged commit b3ae6ba into aquasecurity:main Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants