Skip to content

[proof of concept] tests: eventfd: posix read interface #99231

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[proof of concept] tests: eventfd: posix read interface #99231

wants to merge 1 commit into from
+23 −0

Conversation

@casaroli
Copy link

@casaroli casaroli commented Nov 12, 2025 •
edited
Loading

This test is expected to fail, as a proof that there is a deadlock in the fdtable+eventfd code. After we fix the bug, the test should pass.

The eventfd API should work with the standard POSIX read/write functions with the same behavior as the non-standard eventfd_read/write API.

We add the tests for these interfaces.
@github-actions
Copy link

github-actions bot commented Nov 12, 2025

Hello @casaroli, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊
@casaroli casaroli mentioned this pull request Nov 12, 2025
Open
1 task
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 14, 2025
View reviewed changes
@casaroli
tests: eventfd: posix read interface
603deb5 
The eventfd API should work with the standard POSIX read/write functions with the same behavior as the non-standard eventfd_read/write API. We add the tests for these interfaces. Signed-off-by: Marco Casaroli <marco.casaroli@gmail.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 14, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
E Security Rating on New Code (required ≥ C)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 14, 2025
View reviewed changes
tests/posix/eventfd/src/blocking.c
struct eventfd_fixture *fixture = arg1;
int ret;

ret = read(fixture->fd, &value, 8);

Check failure

Code scanning / SonarCloud

POSIX functions should not be called with arguments that trigger buffer overflows High test

"read" overflows write buffer "&value"; passed size "8" (64) exceeds buffer size (8) See more on SonarQube Cloud
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@casaroli