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

Add heap leak from the linker. Fixes Gallopsled/pwntools#435 #656

Closed
wants to merge 2 commits into from
Closed

Conversation

zachriggle
Copy link
Member

(cherry picked from commit 5808882)

(cherry picked from commit 5808882)
@zachriggle
Copy link
Member Author

zachriggle commented Jul 27, 2016

So this is a regular development feature, which I think per @idolf and I talking should just go in a dev or beta branch somewhere, and not be a pull-request.

@idolf, can you provide guidance?

I appear to have lost the link to the planning document for all this stuff 😦

@TethysSvensson TethysSvensson modified the milestone: 3.1.0 Aug 18, 2016
@TethysSvensson
Copy link
Contributor

Yes, this should go to the dev branch (that does not exist yet). Let us look at it after the release.

@TethysSvensson
Copy link
Contributor

Could you re-open this PR against the dev branch?

Kyle-Kyle pushed a commit to Kyle-Kyle/pwntools that referenced this pull request Apr 25, 2021
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.

2 participants