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.

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

[Coverity CID :190617]Control flow issues in /subsys/net/lib/lwm2m/lwm2m_rw_json.c #12316

Closed
mandarcthorat1 opened this issue Jan 4, 2019 · 2 comments
Closed

[Coverity CID :190617]Control flow issues in /subsys/net/lib/lwm2m/lwm2m_rw_json.c #12316

mandarcthorat1 opened this issue Jan 4, 2019 · 2 comments
Assignees
@mike-scott
Labels
area: LWM2M area: Networking bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix priority: medium Medium impact/importance bug

Comments

@mandarcthorat1
Copy link
Contributor

Static code scan issues seen in File: /subsys/net/lib/lwm2m/lwm2m_rw_json.c
Category: Control flow issues
Function: do_write_op_json
Component: Networking
CID: 190617
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
@galak galak added the priority: medium Medium impact/importance bug label Jan 8, 2019
@mike-scott
Copy link
Contributor

@nashif I have next week scheduled to finish LwM2M socket conversion. Should we still fix this?

@mike-scott
Copy link
Contributor

Fixed in merged commit:
181544b#diff-a4d0f86a3fa08e60a99ec94935e3b6dd

Closing.

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

@mike-scott mike-scott

Labels
area: LWM2M area: Networking bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix priority: medium Medium impact/importance bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mike-scott @galak @mandarcthorat1