m4b

Hmmm, can you provide the binary for repro? Or if you have an idea of how to fix this, PRs welcome :)

Seems reasonable to me, would you like to make a PR adding this? (ideally backwards compatible/non breaking) :)

If you understand the issue and might be willing to fix, a PR would be appreciated :)

>@m4b How do you see this PR? Could you give me what you would like to see to be addressed/directions on how to get this merged? Thank you so much...

interesting; @IridiumXOR would you be interested in working on a PR to fix this? :)

so while reading the PR for fixing this issue, it was revealed that the primary cause of this was that the full file was not being loaded into memory, so...

Unlike e.g., PE or mach-o, elf binaries that "import" a symbol does not have an explicit mapping from symbol -> library providing it (the term for this in mach-o is...

You could perhaps use the GnuHash (although i see it's only constructor is unsafe) to do fast symbol lookups with: https://github.com/m4b/goblin/blob/d7e8e29646157a8c0f9b04337f729be06273b33b/src/elf/gnu_hash.rs#L199 in order to find which library exports a symbol...

Seems correct yes. Since it returns an optional if it is present then it’ll be Some. if you’re asking whether a “delay loaded” import is described by this descriptor I...

Yes this is a bug. Looks like we only tested for both elfs and std https://github.com/m4b/goblin/blob/c81ebdbe5686427090726a5156bd128549ac7499/Makefile#L34 . I’m surprised this hasn’t come up before. Would you be interested in pushing...