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

Merge pull request #2996 from lucas-clemente/remove-unneeded-overflow-check

Browse Source 
remove unneeded overflow check when parsing the max_ack_delay
This commit is contained in:
Marten Seemann
2021-01-15 17:04:01 +08:00
committed by GitHub
parent 460f7883b0 2e5c43bef0
commit b50933737d
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
internal/wire/transport_parameters.go
View File

@@ -303,11 +303,7 @@ func (p *TransportParameters) readNumericTransportParameter(
if val > uint64(protocol.MaxMaxAckDelay/time.Millisecond) {
return fmt.Errorf("invalid value for max_ack_delay: %dms (maximum %dms)", val, protocol.MaxMaxAckDelay/time.Millisecond)
}
maxAckDelay := time.Duration(val) * time.Millisecond
if maxAckDelay < 0 {
maxAckDelay = utils.InfDuration
}
p.MaxAckDelay = maxAckDelay
p.MaxAckDelay = time.Duration(val) * time.Millisecond
case activeConnectionIDLimitParameterID:
p.ActiveConnectionIDLimit = val
case maxDatagramFrameSizeParameterID: