Skip to content

fixed some bugs in the pintool#4

Open
sam-b wants to merge 3 commits intowapiflapi:masterfrom
sam-b:master
Open

fixed some bugs in the pintool#4
sam-b wants to merge 3 commits intowapiflapi:masterfrom
sam-b:master

Conversation

@sam-b
Copy link
Contributor

@sam-b sam-b commented May 5, 2015

Fixed formatting and start up memory allocation issues.

@BestPig
Copy link
Contributor

BestPig commented May 5, 2015

Record variable should be set to true if the binary has no main symbol otherwise stripped binary cannot be monitored properly.

On OS X, the symbol is _main instead of main like others.

@sam-b
Copy link
Contributor Author

sam-b commented May 5, 2015

Good point on the first - will edit tomorrow morning and add in the OSX option.

@sam-b
Copy link
Contributor Author

sam-b commented May 6, 2015

@BestPig Updated the pull request to fix these- thanks :)

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