Skip to content

[Coverity CID :188888]Incorrect expression in /tests/kernel/interrupt/src/nested_irq.c #10752

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
Closed
mandarcthorat1 opened this issue Oct 22, 2018 · 1 comment
Closed

[Coverity CID :188888]Incorrect expression in /tests/kernel/interrupt/src/nested_irq.c #10752

mandarcthorat1 opened this issue Oct 22, 2018 · 1 comment
Assignees
@spoorthik
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 Coverity A Coverity detected issue or its fix priority: low Low impact/importance bug

Comments

@mandarcthorat1
Copy link
Contributor

Static code scan issues seen in File: /tests/kernel/interrupt/src/nested_irq.c
Category: Incorrect expression
Function: isr0
Component: Tests
CID: 188888
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996
@mandarcthorat1 mandarcthorat1 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 Coverity A Coverity detected issue or its fix labels Oct 22, 2018
@nashif nashif added the priority: low Low impact/importance bug label Nov 2, 2018
@spoorthik spoorthik self-assigned this Dec 10, 2018
@spoorthik
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spoorthik spoorthik

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 Coverity A Coverity detected issue or its fix priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nashif @spoorthik @mandarcthorat1