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

elf_dump is not thread-safe ;) #7

Open
br0ns opened this issue Sep 11, 2015 · 3 comments
Open

elf_dump is not thread-safe ;) #7

br0ns opened this issue Sep 11, 2015 · 3 comments

Comments

@br0ns
Copy link

br0ns commented Sep 11, 2015

No description provided.

@br0ns
Copy link
Author

br0ns commented Sep 11, 2015

Possible fix: add __thread before fdesc in elf.c. More info: TLS.

@athas
Copy link

athas commented Sep 11, 2015

That would not be C code.

@oleks
Copy link
Member

oleks commented Sep 11, 2015

It was meant to be so that ARK-students can understand elf_dump by the end of the ARK course. This will be relevant to fix before CompSys though 👍

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

No branches or pull requests

3 participants