Fix #391: Add missing buffer null check by xerial · Pull Request #392 · msgpack/msgpack-java

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@xerial

Copy link Copy Markdown

Member

@xerial xerial commented

Sep 30, 2016

edited

Loading

If write/addPayload is used without any preceding packXXX methods, MessagePacker throws NPE #391

@frsyuki

Copy link Copy Markdown

Member

frsyuki commented

Sep 30, 2016

LGTM!

@xerial

Copy link Copy Markdown

Member Author

xerial commented

Sep 30, 2016

Thanks for checking.

@xerial xerial merged commit 0c0c41f into develop

Sep 30, 2016

@xerial xerial deleted the add-buffer-null-check branch

September 30, 2016 22:04

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@xerial @frsyuki