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

Missing constructor symbols in elf #149

Closed
jayvdb opened this issue Nov 9, 2019 · 3 comments · Fixed by #161
Closed

Missing constructor symbols in elf #149

jayvdb opened this issue Nov 9, 2019 · 3 comments · Fixed by #161

Comments

@jayvdb
Copy link
Contributor

jayvdb commented Nov 9, 2019

When I build some Python packages, and then run object's nm on the generated .o files, some symbols are missing

e.g. https://pypi.org/project/pyzmq/ , https://pypi.org/project/gevent/

> nm build/temp.linux-x86_64-3.7/zmq/backend/cython/message.o | grep __pyx_wrapperbase_3zmq_7backend_6cython_7message_5Frame_2__init__
0000000000000038 C __pyx_wrapperbase_3zmq_7backend_6cython_7message_5Frame_2__init__
> ~/projects/rust/object/target/debug/examples/nm build/temp.linux-x86_64-3.7/zmq/backend/cython/message.o | grep __pyx_wrapperbase_3zmq_7backend_6cython_7message_5Frame_2__init__
0000000000000020 0000000000000038 U __pyx_wrapperbase_3zmq_7backend_6cython_7message_5Frame_2__init__

This is blocking indygreg/PyOxidizer#183

https://pypi.org/project/psutil/ appears to be a different problem.

@jayvdb jayvdb changed the title Missing symbols Missing constructor symbols Nov 9, 2019
@philipc
Copy link
Contributor

philipc commented Nov 9, 2019

Need to add support for common symbols (SHN_COMMON), both for reading and writing.

@jayvdb
Copy link
Contributor Author

jayvdb commented Nov 9, 2019

I see it at https://github.com/m4b/goblin/blob/495475b9be1ca8d1df0ce40d7dceffcae9cdede4/src/elf/section_header.rs#L80

Is that enough in goblin, or does it also need an enhancement?

@philipc
Copy link
Contributor

philipc commented Nov 10, 2019

That's the definition, but that by itself doesn't do anything. It also needs to be handled during reading and writing.

@jayvdb jayvdb changed the title Missing constructor symbols Missing constructor symbols in elf and mach-o Nov 27, 2019
@jayvdb jayvdb changed the title Missing constructor symbols in elf and mach-o Missing constructor symbols in elf Nov 27, 2019
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 a pull request may close this issue.

2 participants