Skip to content

tests: run nss-{users|hosts} by default#21778

Merged
yuwata merged 2 commits intosystemd:mainfrom
evverx:test-cifuzz
Jan 4, 2022
Merged

tests: run nss-{users|hosts} by default#21778
yuwata merged 2 commits intosystemd:mainfrom
evverx:test-cifuzz

Conversation

@evverx
Copy link
Copy Markdown
Contributor

@evverx evverx commented Dec 14, 2021

to prevent issues like #21935 by testing the code under ASan/UBSan

@evverx evverx changed the title make sure CIFuzz can find "affected" fuzz targets tests: run nss-{users|hosts} by default Dec 31, 2021
to prevent issues like #21935
by testing the code under ASan/UBSan
Those libraries aren't provided by systemd so they shouldn't be
included here
@evverx evverx force-pushed the test-cifuzz branch 2 times, most recently from aa1691b to 951280c Compare January 3, 2022 17:23
@evverx evverx marked this pull request as ready for review January 3, 2022 21:44
@evverx
Copy link
Copy Markdown
Contributor Author

evverx commented Jan 3, 2022

@yuwata looks like the tests seem to be passing everywhere now that #21975 is gone. Could you take a look?

Copy link
Copy Markdown
Member

@yuwata yuwata left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@yuwata yuwata merged commit 1341f25 into systemd:main Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants