lwIP - A Lightweight TCP/IP stack - Bugs: bug #27215, TCP sent() callback gives leagin...
You are not allowed to post comments on this tracker with your current authentication level.
bug #27215: TCP sent() callback gives leagin and trailing 1 byte len (SYN/FIN)
Submitter: | Chris N. Strahm <cstrahm> | ||
Submitted: | Mon 10 Aug 2009 01:34:20 PM UTC | ||
Category: | TCP | Severity: | 3 - Normal |
Item Group: | Feature Request | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | 1.3.0 |
Jump to the original submission
Wed 21 Oct 2009 03:45:23 PM UTC, comment #7: |
Simon Goldschmidt <goldsimon> |
Wed 21 Oct 2009 03:17:35 PM UTC, comment #6: D'oh, didn't think of that. Your objection is of course correct.
|
Simon Goldschmidt <goldsimon> |
Wed 21 Oct 2009 11:37:24 AM UTC, comment #5: I think I have an objection! Sorry for the delay. I don't quite understand the new "if ()" block.
|
Kieran Mansley <kieranm> |
Sun 18 Oct 2009 08:31:00 AM UTC, comment #4: Checked in as there were no objections... |
Simon Goldschmidt <goldsimon> |
Fri 16 Oct 2009 02:31:32 PM UTC, comment #3: The attached patch tries to solve this bug by using this condition:
|
Simon Goldschmidt <goldsimon> |
Tue 25 Aug 2009 09:30:43 AM UTC, comment #2: Updated the summary to better describe the problem |
Simon Goldschmidt <goldsimon> |
Sun 23 Aug 2009 12:30:32 PM UTC, comment #1: I have just been debugging this and the 1 byte is of course the FIN being acked. The same problem exists with the initial SYN, which is because the http_sent function always gets called with 2 bytes more than the file size (unless tcp_sent is called to reset the callback too early, of course).
|
Simon Goldschmidt <goldsimon> |
Mon 10 Aug 2009 01:34:20 PM UTC, original submission:
|
Chris N. Strahm <cstrahm> |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
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 11 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2009-10-21 | goldsimon | Open/Closed | Open | Closed | |
Status | In Progress | Fixed | |||
2009-10-21 | kieranm | Open/Closed | Closed | Open | |
Status | Fixed | In Progress | |||
2009-10-18 | goldsimon | Status | In Progress | Fixed | |
Open/Closed | Open | Closed | |||
2009-10-16 | goldsimon | Status | None | In Progress | |
Assigned to | None | goldsimon | |||
2009-10-15 | kieranm | Planned Release | 1.4.0 | 1.3.2 | |
2009-08-25 | goldsimon | Summary | http_sent() gives trailing 1 byte len [raw] | TCP sent() callback gives leagin and trailing 1 byte len (SYN/FIN) | |
2009-08-20 | kieranm | Planned Release | 1.4.0 |
Powered by Savane 3.13-02a9.
Corresponding source code
Fixed again by
a) for SYN, decreasing pcb->acked by one in state SYN_RCVD and
b) for FIN, decreasing pcb->acked by one when removing a segment containing FIN from unsent or unacked