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

Fix crash under linux in Hamsandwich module (bug 6323) #228

Merged
merged 1 commit into from Mar 24, 2015

Conversation

@Arkshine
Copy link
Member

commented Mar 24, 2015

Silly bug where a same pointer was saved 2 times in a list, resulting segfault on cleanup.
Bug appears only now because libc has been updated recently and the magic has disappeared.
Thanks to The Genius (a.k.a Nextra) for his help to narrow down the issue.

Related to #153

@Arkshine Arkshine added the bug label Mar 24, 2015

@Arkshine Arkshine changed the title Fix crash under linux in Hamsandwich module Fix crash under linux in Hamsandwich module (bug 6323) Mar 24, 2015

@Arkshine Arkshine force-pushed the Arkshine:fix/crash-ham branch 2 times, most recently from f5049fb to b6febae Mar 24, 2015

Arkshine
Fix crash under linux in Hamsandwich module
Fix crash under linux in Hamsandwich module
@Nextra

This comment has been minimized.

Copy link
Contributor

commented Mar 24, 2015

Ship it.

Arkshine pushed a commit that referenced this pull request Mar 24, 2015
Vincent Herbet
Merge pull request #228 from Arkshine/fix/crash-ham
Fix crash under linux in Hamsandwich module (bug 6323)

@Arkshine Arkshine merged commit 4d97a24 into alliedmodders:master Mar 24, 2015

2 checks passed

continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Arkshine Arkshine deleted the Arkshine:fix/crash-ham branch Mar 24, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.