-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
[Coverity CID :210610] Argument cannot be negative in tests/posix/eventfd/src/main.c #25739
Comments
@tsvehagen: Can you please look into this? |
Yes I will check it out |
@pfalcon I'm not really sure how to handle this actually. I guess it has to do with |
The usual note is that Coverity report can be a false positive. And reports against tests are of course less serious than the main codebase (the latter really need to be properly investigated). That said, Coverity is usually smart and can pinpoint some non-obvious things. And one can imagine that as any other software, it's also buggy sometimes, and catching that is only more fun. And here it really looks like Coverity outsmarted itself. I assume you have access to Coverity web UI for this issue (login via Github account works). So, it claims:
But what makes it think so? Let's click "show detail" and find out:
So, it kinda thinks that All in all, that's really an interesting case, I didn't yet see such a situation with Coverity, gives insight of what you can expect from it. Resolution is clearly "false positive" (i.e. "ignore" in Coverity UI). (And yeah, leaving to you to double-check, in case I didn't wake up enough and missed something :-D). |
@pfalcon I don't have access, it says 'Access request pending'. I guess it must be accepted by someone in the Zephyr project. |
@tsvehagen: Ok, I marked it as false positive myself. Hopefully your request will be approved soon. Otherwise it's just about sharing load of "adding and maintaining code in Zephyr", I redirect reports for the code you wrote, but feel free to ask for any help. And just to be explicit, reports re: tests aren't high priority, the idea is to have them handled before the next release ;-). |
Static code scan issues found in file:
https://github.com/zephyrproject-rtos/zephyr/tree/4653b4e63f886a50ac7b72f8d47ba2950ab2dd0d/tests/posix/eventfd/src/main.c#L39
Category: Error handling issues
Function:
test_eventfd
Component: Tests
CID: 210610
Details:
Please fix or provide comments in coverity using the link:
https://scan9.coverity.com/reports.htm#v32951/p12996.
Note: This issue was created automatically. Priority was set based on classification
of the file affected and the impact field in coverity. Assignees were set using the CODEOWNERS file.
The text was updated successfully, but these errors were encountered: