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

tetragon: execve map related fixes #370

Merged
merged 2 commits into from
Aug 30, 2022
Merged

Conversation

olsajiri
Copy link
Contributor

@olsajiri olsajiri commented Aug 29, 2022

Adding execve_map related fixes.

Signed-off-by: Jiri Olsa [email protected]

Adding missing error check for data_event_str.

Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri changed the title tetragon: Add missing error check in execve filename retrieval tetragon: execve map related fixes Aug 29, 2022
Currently we create new execve_map entries when we traverse
parents. That could polute the map with entries which might
not get removed in exit tracepoint.

By using execve_map_get_noinit the tracepoint and kprobe sensors
never get to use execve_map_stats map, so removing it from the
loader tests.

Suggested-by: John Fastabend <[email protected]>
Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri force-pushed the exec_fixes branch 2 times, most recently from 66ab57a to 59edfe4 Compare August 29, 2022 20:06
@olsajiri olsajiri marked this pull request as ready for review August 30, 2022 05:20
@olsajiri olsajiri requested a review from a team as a code owner August 30, 2022 05:20
@olsajiri olsajiri requested a review from sharlns August 30, 2022 05:20
@jrfastab jrfastab merged commit 8975c8d into cilium:main Aug 30, 2022
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.

3 participants