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

tests: libc: Update libc testcases to solve coverity issues #34889

Merged
merged 1 commit into from
May 12, 2021

Conversation

yingmingx
Copy link
Collaborator

@yingmingx yingmingx commented May 6, 2021

1. Refine some testcases for testing some libc functions, include
strlen(),strcmp(),strncmp(),memcmp(),memstr(),memcpy(),memmove()
strtol(),strtoul().
2. Change some code to make the code style consistent.

Fixes #33794
Fixes #33810
Fixes #33806

Signed-off-by: Ying ming mingx.ying@intel.com

@yingmingx yingmingx requested a review from nashif as a code owner May 6, 2021 04:17
@github-actions github-actions bot added the area: Tests Issues related to a particular existing or missing test label May 6, 2021
@yingmingx yingmingx changed the title test:libc:fix coverity issues tests: libc: Redesign some testcases May 6, 2021
Copy link
Collaborator Author

@yingmingx yingmingx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, enjia. This code you mentioned was writen by Jian last year and xiaofs improve the coverage on the basis of Jian's code. I redesigned this testcase, now the coverage of memchr(), memcpy(), memmove() is also 100%.

@enjiamai enjiamai changed the title tests: libc: Redesign some testcases tests: libc: Update libc testcases to solve coverity issues May 10, 2021
1. Refine some testcases for testing some libc functions, include
strlen(),strcmp(),strncmp(),memcmp(),memstr(),memcpy(),memmove()
strtol(),strtoul().
2. Change some code to make the code style consistent.

Fixes zephyrproject-rtos#33794
Fixes zephyrproject-rtos#33810
Fixes zephyrproject-rtos#33806

Signed-off-by: Ying ming <mingx.ying@intel.com>
@nashif nashif merged commit 8e69daf into zephyrproject-rtos:master May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Tests Issues related to a particular existing or missing test
Projects
None yet
3 participants