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

Submitted by:  dupengbo <dupengbo>
Submitted on:  Fri 27 Apr 2007 10:01:58 AM UTC  
 
Category: TCPSeverity: 3 - Normal
Item Group: Faulty BehaviourStatus: Wont Fix
Privacy: PublicAssigned to: None
Open/Closed: ClosedPlanned 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>
Project Administrator
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>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 30 Apr 2007 11:06:16 AM UTCkieranmStatusNone=>Wont Fix
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1