-
Notifications
You must be signed in to change notification settings - Fork 58
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
libsys: Add CHERI_LIB_C18N to CFLAGS when appropriate #2217
Conversation
Hmm, looks like the multi-threaded cheribsdtest binaries using c18n failed tests (or maybe crashed without running any tests?) |
Both ABIs crashed with SIGTRAP in the parent |
Looks like the
|
Why is elf_aux_info a stub? How is that ever meant to work? There's useful information in there... |
Oh I see, it's meant to be filtered such that you get the non-trapping version at run time. So why isn't libsys's definition being used here? It works for non-c18n. |
ktrace shows
which causes the filtee to not be loaded. |
I removed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe just drop the first commit and only have the second commit that "fixes" c18n libsys?
Previously, -DCHERI_LIB_C18N was omitted for libsys. This commit also fixes that.
Previously, lib/c18n/libsys's sigaction calls a libc function which in turn calls the RTLD hook. This is broken but was undetected because c18n was accidentally always disabled in the c18n-version of libsys.
No description provided.