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

Fix pthread #558

Merged
merged 4 commits into from Aug 4, 2017

Conversation

Projects
None yet
3 participants
@anon8675309

anon8675309 commented Aug 3, 2017

Fix for issue #549

LiptonB and others added some commits Aug 3, 2017

Make sure _dl_error_catch_tsd points to an allocator function
This function pointer is called during pthreads initialization. We must
initialize it to point to a function (in this case a SimProcedure) that
returns the address of some valid memory.
Hook three more syscalls
These are called during pthreads initialization, and leaving the outputs
unconstrained causes unnecessary path duplication.
adam
Merge branch 'fix-pthreads' into 'changes-for-upstream'
Support for binaries that link pthreads

See merge request !1
@rhelmot

This comment has been minimized.

Member

rhelmot commented Aug 3, 2017

this is very cool! can you please move _dl_initial_error_catch_tsd into procedures/linux_loader like we did with the rest of the loader simprocedures during the refactor?

@rhelmot rhelmot merged commit 813c331 into angr:master Aug 4, 2017

0 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
@rhelmot

This comment has been minimized.

Member

rhelmot commented Aug 4, 2017

Thanks!

@anon8675309 anon8675309 referenced this pull request Oct 17, 2017

Closed

Out of paths #24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment