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

change hooked_syscalls event so users can specify syscalls to check. #3136

Merged
merged 3 commits into from Jun 26, 2023

Conversation

AsafEitani
Copy link
Contributor

related to #2055 and fix some bugs the triggered events.

1. Explain what the PR does

commit 542fb53 (HEAD -> syscall_hooking_change, origin/syscall_hooking_change)
Author: AsafEitani eitaniasaf@gmail.com
Date: Thu May 18 15:12:34 2023 +0300

change hooked_syscalls event so users can specify syscalls to check.
related to #2055 and fix some bugs the triggered events.

2. Explain how to test it

specify syscalls to check by adding -f hooked_syscalls.args.check_syscalls=<syscall>,<syscall> for example:dist/tracee-ebpf -f e=hooked_syscalls -f hooked_syscalls.args.check_syscalls=execve,kill,getdents,getdents64`

see the list of checked syscalls and the list of hooked syscalls.
load a diamorphine kernel module rootkit and see the new event popping up with 3 hooked syscalls.

3. Other comments

Some of the changes were made to other events to fix bugs.

@AsafEitani AsafEitani force-pushed the syscall_hooking_change branch 2 times, most recently from 8ecf5af to ff3ec72 Compare May 24, 2023 11:41
@AsafEitani AsafEitani marked this pull request as draft May 24, 2023 15:56
@AsafEitani AsafEitani force-pushed the syscall_hooking_change branch 2 times, most recently from eb1cb1f to 4cbbc4f Compare May 28, 2023 12:53
@AsafEitani AsafEitani force-pushed the syscall_hooking_change branch 2 times, most recently from 985d973 to 0f61ccb Compare May 29, 2023 10:29
@AsafEitani AsafEitani marked this pull request as ready for review May 29, 2023 11:26
@AsafEitani AsafEitani added this to the v0.16.0 milestone Jun 15, 2023
docs/docs/events/builtin/extra/hooked_syscalls.md Outdated Show resolved Hide resolved
docs/docs/events/builtin/extra/hooked_syscalls.md Outdated Show resolved Hide resolved
pkg/ebpf/c/common/buffer.h Show resolved Hide resolved
pkg/ebpf/c/common/buffer.h Show resolved Hide resolved
pkg/ebpf/c/common/consts.h Outdated Show resolved Hide resolved
pkg/ebpf/c/common/consts.h Outdated Show resolved Hide resolved
pkg/ebpf/capture.go Outdated Show resolved Hide resolved
pkg/ebpf/tracee.go Outdated Show resolved Hide resolved
pkg/ebpf/tracee.go Outdated Show resolved Hide resolved
pkg/events/derive/hooked_syscall.go Outdated Show resolved Hide resolved
pkg/ebpf/c/common/buffer.h Show resolved Hide resolved
pkg/events/derive/hooked_syscall.go Outdated Show resolved Hide resolved
pkg/ebpf/tracee.go Outdated Show resolved Hide resolved
pkg/ebpf/capture.go Outdated Show resolved Hide resolved
@AsafEitani AsafEitani force-pushed the syscall_hooking_change branch 3 times, most recently from 57985c2 to 68276d4 Compare June 25, 2023 10:00
@yanivagman yanivagman merged commit 637b65a into aquasecurity:main Jun 26, 2023
25 checks passed
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