-
Notifications
You must be signed in to change notification settings - Fork 87
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
Remove getenv calls before libc startup #780
Comments
github-actions
bot
added
the
needs-triage
Bug does not yet have a priority assigned
label
Aug 13, 2020
davidchisnall
added this to Remove musl dependencies from enclave / lkl init
in Clean up layering
Aug 17, 2020
bodzhang
added
p1
Medium priority
and removed
needs-triage
Bug does not yet have a priority assigned
labels
Aug 21, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
lkl_start_init is using getenv fairly early in the initialization sequence. Which is requiring an early call to
__init_libc
.According to @davidchisnall -
When we parse the environment variables from the config file, we should pull the SGX-LKL ones into globals and skip them, then we wouldn't need that extra complexity here.
From - lsds/sgx-lkl-musl#18 (comment)
The text was updated successfully, but these errors were encountered: