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

uninitialized index member of the utest_test_state_t structure #49

Closed
DBJDBJ opened this issue Jan 11, 2021 · 3 comments · Fixed by #50
Closed

uninitialized index member of the utest_test_state_t structure #49

DBJDBJ opened this issue Jan 11, 2021 · 3 comments · Fixed by #50

Comments

@DBJDBJ
Copy link

DBJDBJ commented Jan 11, 2021

image

After line #644 the assignment to the index member is what might be mising?

    utest_state.tests[index].func = &utest_##SET##_##NAME;       \
    utest_state.tests[index].name = name;                                      \
    utest_state.tests[index].index = index;                                      \
@DBJDBJ DBJDBJ changed the title uninit var uninitialized index member of the utest_test_state_t structure Jan 11, 2021
@DBJDBJ DBJDBJ closed this as completed Jan 11, 2021
@DBJDBJ DBJDBJ reopened this Jan 11, 2021
@sheredom
Copy link
Owner

Good find! I'll do a PR 😄

sheredom added a commit that referenced this issue Jan 12, 2021
@DBJDBJ
Copy link
Author

DBJDBJ commented Jan 12, 2021

Roger that @sheredom

Also the line #627 (see above) might be safely removed?

@sheredom
Copy link
Owner

I think line 627 needs to stay incase the UTEST is in a file all on its own.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants