gr1ffon/quic-go
1
0
Fork 0
forked from quic-go/quic-go
Code Pull Requests Activity

remove the internal error for MAX_STREAM_DATA frames for closed streams

Browse Source
This commit is contained in:
Marten Seemann
2017-12-10 23:23:27 +07:00
parent 489ea7fa1a
commit ba4e980876
1 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
session.go
View File

@@ -32,8 +32,7 @@ type receivedPacket struct {
}
var (
errRstStreamOnInvalidStream = errors.New("RST_STREAM received for unknown stream")
errWindowUpdateOnClosedStream = errors.New("WINDOW_UPDATE received for an already closed stream")
errRstStreamOnInvalidStream = errors.New("RST_STREAM received for unknown stream")
)
var (
@@ -551,8 +550,6 @@ func (s *session) handleFrames(fs []wire.Frame, encLevel protocol.EncryptionLeve
case errRstStreamOnInvalidStream:
// Can happen when RST_STREAMs arrive early or late (?)
utils.Errorf("Ignoring error in session: %s", err.Error())
case errWindowUpdateOnClosedStream:
// Can happen when we already sent the last StreamFrame with the FinBit, but the client already sent a WindowUpdate for this Stream
default:
return err
}
@@ -597,7 +594,8 @@ func (s *session) handleMaxStreamDataFrame(frame *wire.MaxStreamDataFrame) error
return err
}
if str == nil {
return errWindowUpdateOnClosedStream
// stream is closed and already garbage collected
return nil
}
str.UpdateSendWindow(frame.ByteOffset)
return nil