Skip to content

[Coverity CID: 183059] Memory - corruptions in /samples/net/mbedtls_dtlsclient/src/dtls_client.c #6673

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
Shilpashree3lc-zz opened this issue Mar 19, 2018 · 1 comment
Labels
area: Samples Samples 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

@Shilpashree3lc-zz
Copy link
Contributor

Static code scan issues seen in File: /samples/net/mbedtls_dtlsclient/src/dtls_client.c
Category: Memory - corruptions
Function: dtls_client
Component: Samples
CID: 183059
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: Samples Samples bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix labels Mar 19, 2018
@MaureenHelm MaureenHelm added the priority: low Low impact/importance bug label Mar 23, 2018
@sritej20
Copy link
Contributor

Can be closed as they are false positives same as #6695 , #7509 , #7510 . The structs can have different sizes only if different config files are loaded for the test tool and the source code. Since this cant be possible as the CONFIG_MBEDTLS_CFG_FILE="config-***.h" is passed the prj.conf file of each of the samples code, which in turn is used to compile the source code , the struct's will have the same size as when declared and memset .

@nashif nashif closed this as completed Jul 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Samples Samples 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
Development

No branches or pull requests

4 participants