buglwIP - A Lightweight TCP/IP stack - Bugs: bug #19722, when incoming segment is...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #19722: when incoming segment is out-of-sequence, recv_wnd should be update too

Submitter:  dupengbo <dupengbo>
Submitted:  Fri 27 Apr 2007 10:01:58 AM UTC
   
 
Category:  TCP Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Wont Fix
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Planned Release:  None
lwIP version:  None

Sat 28 Apr 2007 02:36:44 PM UTC, comment #3: 

hello Kieran Mansley :
I have tested the issue, you are right,
thank you very much:)

dupengbo <dupengbo>
Fri 27 Apr 2007 02:25:06 PM UTC, comment #2: 

Maybe you are right, it is unecessary,
But I will do some test to verify it.

Thank you very much!

dupengbo <dupengbo>
Fri 27 Apr 2007 10:34:18 AM UTC, comment #1: 

I'm not so sure.  Can you explain why you think we should make the recv window small when queueing out of order segments?

The recv window tells the other end how much it can send (relative to the ACK in the packet) to us.  When we get out of order data we don't ACK it, so the ACK doesn't change.  If we made the recv window smaller, we would be telling the other end it can send less than it could before, which is rather naughty and seems unecessary.  We should still be able to accept the same amount of data regardless of the order we receive it in.

If memory is so tight that you want to advertise a smaller window because some has been taken up buffering the out of order packet, I think a better solution would be to turn off queueing of the out of order segments, which would make lwIP discard them and force the other end to retransmit in-order.

However, if you can explain why, I'm happy to change my mind.

Kieran Mansley <kieranm>
Group Member
Fri 27 Apr 2007 10:01:58 AM UTC, original submission:  

If the incoming segment has data that is above the next sequence number expected, recv window should be update too.

In LWIP, there is no produce for this case.

Because, these data is saved in ooseg(Received out of sequence segments), has used the receive's space, the recv window should be updated.

dupengbo <dupengbo>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by kieranm (Posted a comment)
  • -email is unavailable- added by dupengbo (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2007-04-30 kieranm StatusNone Wont Fix
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code