Skip to content

Commit

Permalink
msg/json: fix panic error when length is invalid
Browse files Browse the repository at this point in the history
  • Loading branch information
fatedier committed May 21, 2018
1 parent 354693c commit 674e937
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions msg/json/process.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
var (
ErrMsgType = errors.New("message type error")
ErrMaxMsgLength = errors.New("message length exceed the limit")
ErrMsgLength = errors.New("message length error")
ErrMsgFormat = errors.New("message format error")
)

Expand All @@ -46,6 +47,9 @@ func (msgCtl *MsgCtl) readMsg(c io.Reader) (typeByte byte, buffer []byte, err er
if length > msgCtl.maxMsgLength {
err = ErrMaxMsgLength
return
} else if length < 0 {
err = ErrMsgLength
return
}

buffer = make([]byte, length)
Expand Down

0 comments on commit 674e937

Please sign in to comment.