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 issue seen with CID: 178064 #4397

Closed
Shilpashree3lc-zz opened this issue Oct 18, 2017 · 2 comments
Closed

Coverity issue seen with CID: 178064 #4397

Shilpashree3lc-zz opened this issue Oct 18, 2017 · 2 comments
Assignees
@rveerama1
Labels
bug The issue is a bug, or the PR is fixing a bug priority: high High impact/importance bug

Comments

@Shilpashree3lc-zz
Copy link
Contributor

In File: /tests/net/lib/coap/src/main.c
Category: Memory - corruptions
Function: test_block_size
Component: Tests
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996
@nashif nashif added the bug The issue is a bug, or the PR is fixing a bug label Oct 18, 2017
@jukkar jukkar mentioned this issue Oct 18, 2017
Merged
@andrewboie
Copy link
Contributor

would it be possible to give more descriptive names to these bugs?
like include the path to the affected file in the subject line

jukkar added a commit to jukkar/zephyr that referenced this issue Oct 19, 2017
@jukkar
tests: net: coap: Use proper array for payload
347fa36 
The payload variable was one byte long but we read 32 bytes
from it.

Coverity-CID: 178064
Fixes zephyrproject-rtos#4397

Signed-off-by: Jukka Rissanen <[email protected]>
@linkmeyer linkmeyer added the priority: high High impact/importance bug label Oct 23, 2017
@lpereira
Copy link
Collaborator

lpereira commented Oct 23, 2017 •
edited
Loading

Fix in #4413 is correct.

jukkar added a commit that referenced this issue Oct 24, 2017
@jukkar
tests: net: coap: Use proper array for payload
964f399 
The payload variable was one byte long but we read 32 bytes
from it.

Coverity-CID: 178064
Fixes #4397

Signed-off-by: Jukka Rissanen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rveerama1 rveerama1

Labels
bug The issue is a bug, or the PR is fixing a bug priority: high High impact/importance bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@lpereira @nashif @andrewboie @linkmeyer @rveerama1 @Shilpashree3lc-zz