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 nick->host == NULL crash #529

Merged
merged 1 commit into from Aug 25, 2016

Conversation

Projects
None yet
2 participants
@ailin-nemui
Copy link
Contributor

commented Aug 22, 2016

may resolve issue #500

@dequis

This comment has been minimized.

Copy link
Member

commented Aug 25, 2016

Welp, I give up, can't figure out how to reproduce this path. I got a lot of assertion warnings of missing hosts, but no idea how to get the first host to exist and the printnick one to be null.

Either way this does look like something that would fix the issue of the other ticket, so LGTM

@ailin-nemui

This comment has been minimized.

Copy link
Contributor Author

commented Aug 25, 2016

thanks for trying

@ailin-nemui ailin-nemui merged commit c8630ac into irssi:master Aug 25, 2016

1 check passed

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

@ailin-nemui ailin-nemui deleted the ailin-nemui:issue500 branch Aug 25, 2016

ailin-nemui added a commit to ailin-nemui/irssi that referenced this pull request Sep 21, 2016

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