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.

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

MISRA C - Review the use of memcpy, memcmp and memmove #10013

Closed
ceolin opened this issue Sep 16, 2018 · 2 comments
Closed

MISRA C - Review the use of memcpy, memcmp and memmove #10013

ceolin opened this issue Sep 16, 2018 · 2 comments
Labels
area: MISRA-C bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Milestone

Comments

@ceolin
Copy link
Member

ceolin commented Sep 16, 2018

The usage of these functions should be reviewed to ensure that they are being called with parameters of compatible type. The following snippet is an example of a violation:

void example(char a[3], int b[3]) {
    (void) memcpy(a, b, 3); /* -> violation */
}

part of #9552

@ceolin ceolin added Feature A planned feature with a milestone priority: medium Medium impact/importance bug area: MISRA-C labels Sep 16, 2018
@ceolin ceolin added this to the v1.14.0 milestone Sep 16, 2018
@ceolin
Copy link
Member Author

ceolin commented Sep 26, 2018

rules 8.14, 19.1 and 17.7

@ceolin ceolin added bug The issue is a bug, or the PR is fixing a bug and removed Feature A planned feature with a milestone labels Feb 8, 2019
@nashif nashif removed Feature A planned feature with a milestone labels Feb 21, 2019
@ceolin
Copy link
Member Author

ceolin commented Feb 25, 2019

There is no fault remaining in Zephyr's kernel.

@ceolin ceolin closed this as completed Feb 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: MISRA-C bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Development

No branches or pull requests

2 participants