[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lwip-devel] [bug #26267] Simultaneous close

From: Oleg Tyshev
Subject: [lwip-devel] [bug #26267] Simultaneous close
Date: Fri, 24 Apr 2009 11:59:42 +0000
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv: Gecko/2009040821 Firefox/3.0.9 GTB5

Follow-up Comment #18, bug #26267 (project lwip):

In reply to comment #13.
I think it would be better get ride of snd_max.
SND_NXT is used in all RFCs.
Why snd_nxt is overwritten by retransmission?
By retransmission segment is moved from unacked queue to unsent queue.


pcb->snd_nxt = ntohl(seg->tcphdr->seqno) + TCP_TCPLEN(seg);
if (TCP_SEQ_LT(pcb->snd_max, pcb->snd_nxt)) {
   pcb->snd_max = pcb->snd_nxt;

we should make something like this

tmp_snd_nxt = ntohl(seg->tcphdr->seqno) + TCP_TCPLEN(seg);
if (TCP_SEQ_LT(pcb->snd_nxt, tmp_snd_nxt)) {
  pcb->snd_nxt = tmp_snd_nxt;

tcp_rexmit_rto() and tcp_rexmit() shouldn't modify snd_nxt.


Reply to this item at:


  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]