Open ScriptSathi opened 6 days ago
Name | Link |
---|---|
Latest commit | 1fb7dbca69ced745a76f9f0ca36c5e39ff0dc4a3 |
Latest deploy log | https://app.netlify.com/sites/tetragon/deploys/674216bc2d27200008186dd3 |
Deploy Preview | https://deploy-preview-3143--tetragon.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
The discussion for this PR can be found here https://github.com/cilium/tetragon/issues/3142
This is currently a draft. I wanted to start the discussion before submitting the final code, as I think, it is a big enough PR.
Take this PR in the today state as a proof of concept for dynamic parameter extraction. I will continue to work on this PR until the below checks are done.
At the current state, the PR is able to
Description
This PR introduce the dynamic parameter extraction
Comments
OverwriteType
parameter. But since the functionargSelectorType
does not receiveEventConfig
, it is not possible to search for the type using directly BTF types. So I suggest doing another PR before this one is merged to do so if possible. Then I'll remove the parameter. It is also not possible to add an if condition foruprobes
in this function. So if the user defines the parameter, it will overwrite the type at this moment.u8
to store the offset, as the verifier does not allow me to useu16
. At this moment, I don't know if it is possible to found offset > 255 in BTF structures. For my uses cases, usingu8
was enough.Test the PR
You can use the following config
If you want to test it with more arguments, you can use
bprm_creds_from_file
hook. It hasstruct linux_binprm
andstruct file
which are supported.