diff options
| author | Adin Scannell <ascannell@google.com> | 2018-08-08 22:22:40 -0700 |
|---|---|---|
| committer | Shentubot <shentubot@google.com> | 2018-08-08 22:23:47 -0700 |
| commit | 48b5b35b2bd46ecd043f95d5f470da71046af760 (patch) | |
| tree | 393d8558b65a73ccd5bb59f8ad0648d181cd2803 /pkg/p9/transport.go | |
| parent | ea1e39a314d3a248d8b682a9f63e686530597d61 (diff) | |
Fix error handling for bad message sizes.
The message size check is legitimate: the size must be negotiated, which
relies on the fixed message limit up front. Sending a message larger than
that indicates that the connection is out of sync and is considered a
socket error (disconnect).
Similarly, sending a size that is too small indicates that the stream is
out-of-sync or invalid.
PiperOrigin-RevId: 207996551
Change-Id: Icd8b513d5307e9d5953dbb957ee70ceea111098d
Diffstat (limited to 'pkg/p9/transport.go')
| -rw-r--r-- | pkg/p9/transport.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pkg/p9/transport.go b/pkg/p9/transport.go index c42b41fcf..b5df29961 100644 --- a/pkg/p9/transport.go +++ b/pkg/p9/transport.go @@ -206,11 +206,11 @@ func recv(s *unet.Socket, msize uint32, lookup lookupTagAndType) (Tag, message, // The message is too small. // // See above: it's probably screwed. - return NoTag, nil, ErrNoValidMessage + return NoTag, nil, ErrSocket{ErrNoValidMessage} } if size > maximumLength || size > msize { // The message is too big. - return NoTag, nil, &ErrMessageTooLarge{size, msize} + return NoTag, nil, ErrSocket{&ErrMessageTooLarge{size, msize}} } remaining := size - headerLength |