fix(driver): hoist flags declaration to the top of f_sys_accept4_x#2994
Conversation
This prevents some build failures if the compilation toolchain uses pedantic flags. Signed-off-by: Leonardo Di Giovanna <leonardodigiovanna1@gmail.com>
|
Please double check driver/SCHEMA_VERSION file. See versioning. /hold |
|
Testing against kernel/distro matrix: https://github.com/falcosecurity/libs/actions/runs/25742128518/job/75595174915 |
False positive. I'm gonna unhold once the CI is green |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ekoops, irozzo-1A, leogr, therealbobo The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2994 +/- ##
=======================================
Coverage 75.73% 75.73%
=======================================
Files 299 299
Lines 33164 33164
Branches 5168 5168
=======================================
Hits 25116 25116
Misses 8048 8048
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
X64 Matrix
ARM64 Matrix
|
|
/hold cancel |
What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area driver-kmod
Does this PR require a change in the driver versions?
What this PR does / why we need it:
This prevents some build failures if the compilation toolchain uses pedantic flags.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/milestone 10.1.0+driver
Does this PR introduce a user-facing change?: