Skip to content

[Coverity CID :190377]Control flow issues in /sanitylog/nrf52840_pca10056/samples/net/echo_server/test_nrf_openthread/zephyr/ext_proj/Source/ot/examples/platforms/utils/settings_flash.c #12325

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

Closed
mandarcthorat1 opened this issue Jan 4, 2019 · 2 comments
Labels
area: Networking bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix

Comments

@mandarcthorat1
Copy link
Contributor

Static code scan issues seen in File: /sanitylog/nrf52840_pca10056/samples/net/echo_server/test_nrf_openthread/zephyr/ext_proj/Source/ot/examples/platforms/utils/settings_flash.c
Category: Control flow issues
Function: swapSettingsBlock
Component: Networking
CID: 190377
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: Networking bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix labels Jan 4, 2019
@nashif nashif closed this as completed Jan 4, 2019
@nashif
Copy link
Member

nashif commented Jan 4, 2019

you are doing something wrong here, we should not report anything under /sanitylog/. Originally this was ignored in coverity.

@mandarcthorat1
Copy link
Contributor Author

Fixed. We should not encounter these issues from next scan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Networking bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix
Projects
None yet
Development

No branches or pull requests

2 participants