Skip to content
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: 178785] Incorrect expression in /tests/kernel/static_idt/src/static_idt.c #4789

Closed
Shilpashree3lc-zz opened this issue Nov 8, 2017 · 1 comment
Assignees
Labels
area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@Shilpashree3lc-zz
Copy link
Contributor

Static code scan issues seen in File: /tests/kernel/static_idt/src/static_idt.c
Category: Incorrect expression
Function: testing_static_idt
Component: Tests
CID: 178785
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

@Shilpashree3lc-zz Shilpashree3lc-zz added area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug labels Nov 8, 2017
@Shilpashree3lc-zz Shilpashree3lc-zz changed the title [Coverity CID:178785] Incorrect expression in /tests/kernel/static_idt/src/static_idt.c [Coverity CID: 178785] Incorrect expression in /tests/kernel/static_idt/src/static_idt.c Nov 10, 2017
@andrewboie
Copy link
Contributor

It's complaining because a volatile variable is being read in a zassert() function. This is a false positive, zasserts are not conditionally compiled and there are no side effects even if it was. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Development

No branches or pull requests

2 participants