Skip to content

tests: skip the SIGEV_THREAD code path in test_privcand under TSAN#1811

Merged
michalvasko merged 1 commit into
CESNET:develfrom
jktjkt:bug-1810
Jun 4, 2026
Merged

tests: skip the SIGEV_THREAD code path in test_privcand under TSAN#1811
michalvasko merged 1 commit into
CESNET:develfrom
jktjkt:bug-1810

Conversation

@jktjkt
Copy link
Copy Markdown
Contributor

@jktjkt jktjkt commented Jun 3, 2026

This particular check leads to a code path in netopeer2-server which starts a background thread via the C library, and that causes a segfault in TSAN's internals. Yay.

Fixes: #1810
Bug: google/sanitizers#1612

Copy link
Copy Markdown
Member

@michalvasko michalvasko left a comment

Choose a reason for hiding this comment

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

Okay, I suppose, because it only concerns a test. But gcc requires also defined(__has_feature), I think.

@jktjkt
Copy link
Copy Markdown
Contributor Author

jktjkt commented Jun 3, 2026

Okay, I suppose, because it only concerns a test.

Yeah, I opted for a smaller fix than disabling that test completely (which is already done for some other tests).

But gcc requires also defined(__has_feature), I think.

Done, I hope.

This particular check leads to a code path in netopeer2-server which
starts a background thread via the C library, and that causes a segfault
in TSAN's internals. Yay.

Fixes: CESNET#1810
Bug: google/sanitizers#1612
@michalvasko michalvasko merged commit 8b7b59f into CESNET:devel Jun 4, 2026
7 checks passed
@jktjkt jktjkt deleted the bug-1810 branch June 4, 2026 08:44
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 this pull request may close these issues.

2 participants