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

Sections in segment: merge from master #24

Open
wants to merge 23 commits into
base: sections-in-segment
Choose a base branch
from

Conversation

gilescope
Copy link

I've implemented unsupported EhFrame and EhFrameHdr types as unsupported wasn't in the branch. The rest pretty much merged fine.

fitzgen and others added 23 commits August 21, 2017 16:56
Add a function on `SharedLibrary` to go from an AVMA to an SVMA
Clean up panic propagation in linux implementation
...and replace deprecated bindgen methods with the non-deprecated versions.
…-org

Updte links to point to gimli-rs/findshlibs instead of fitzgen/findshlibs
Add support for fetching UUIDs from macho libs
Added a default implementation for unsupported platforms
@philipc
Copy link
Contributor

philipc commented Aug 5, 2018

Can you rebase so that it's only the new commits in the PR?

@philipc
Copy link
Contributor

philipc commented Aug 5, 2018

Oh, I see this is a PR against the sections-in-segment branch, is that what you intended?

@gilescope
Copy link
Author

gilescope commented Aug 5, 2018 via email

@philipc
Copy link
Contributor

philipc commented Aug 5, 2018

Yes, just bring that branch up to date.

Ok. I think it's better to rebase and create your own branch instead. I prefer if the final PR against master doesn't have merge commits in it.

I need to review the linux builds as something is clearly not happy.

The EhFrame trait is hard/impossible to implement on linux, so that'll be one reason why @fitzgen never merged this to master. Note that this inability to find eh_frame is also the reason for this hack in unwind-rs.

For linux, I have an idea that since eh_frame_hdr tells us where eh_frame starts, maybe we can use that to find the segment containing eh_frame, and deduce the length from that. I'll look into this later.

@philipc
Copy link
Contributor

philipc commented Aug 6, 2018

gimli-rs/unwind-rs#18 shows what I was meaning for finding eh_frame, although it's not as good as I wanted.

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 this pull request may close these issues.

5 participants