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
[CLEANUP] Drivers inconsistencies #902
Comments
Issues go stale after 90d of inactivity. Mark the issue as fresh with Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with Provide feedback via https://github.com/falcosecurity/community. /lifecycle stale |
/remove-lifecycle stale |
@Andreagit97 any thoughts on |
uhm I agree with @loresuso one day or another we should convert all |
ok |
The CHECK_RES issue should be marked as completed like this |
The syscall_arg_t should be marked as completed as well @Andreagit97 :) |
@incertum Can you close this issue? |
Thanks for your work @ecbadeaux ! |
/milestone next-driver |
While working on our drivers I am noticing many inconsistencies.
For instance:
s32
andint32_t
types. I think we should use one or the other for consistency, and I would use the_t
ones because are defined by the C standard.CHECK_RES
that can be used to check if a previous operation succeeded or not. It is not consistently used in the whole file.syscall_arg_t
type that is seldom used throughout the file.Still adding things to this. Once we are done with introducing syscall for modern probe we can think about this!
The text was updated successfully, but these errors were encountered: