status-im
/
nim-eth
mirror of https://github.com/status-im/nim-eth.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Differentiate log for invalid packet seqnr without reorder buffer and not (#723)

This commit is contained in:
Kim De Mey 2024-08-29 18:15:26 +02:00 committed by GitHub
parent 98b2a34075
commit b874e12516
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
eth/utp/utp_socket.nim
View File

@ -1212,9 +1212,7 @@ proc processPacketInternal(socket: UtpSocket, p: Packet) =
# with a todo. Currently the reference implementation is follow and packets # with a todo. Currently the reference implementation is follow and packets
# are not resend in this case. # are not resend in this case.
debug "Packet state variables", debug "Packet state variables", pastExpected, acks
pastExpected = pastExpected,
acks = acks
# If packet is totally off the mark, short-circuit the processing # If packet is totally off the mark, short-circuit the processing
if pastExpected >= reorderBufferMaxSize: if pastExpected >= reorderBufferMaxSize:
@ -1230,10 +1228,11 @@ proc processPacketInternal(socket: UtpSocket, p: Packet) =
pastExpected >= (int(uint16.high) + 1) - reorderBufferMaxSize pastExpected >= (int(uint16.high) + 1) - reorderBufferMaxSize
if (isPossibleDuplicatedOldPacket and p.header.pType != ST_STATE): if (isPossibleDuplicatedOldPacket and p.header.pType != ST_STATE):
debug "Invalid packet sequence number, within reorder buffer",
pastExpected
socket.sendAck() socket.sendAck()
else:
debug "Got an invalid packet sequence number, too far off", debug "Invalid packet sequence number, too far off", pastExpected
pastExpected = pastExpected
return return
var (ackedBytes, minRtt) = var (ackedBytes, minRtt) =