lwIP - A Lightweight TCP/IP stack - Bugs: bug #21492, TCP MSS can be larger than MTU
You are not allowed to post comments on this tracker with your current authentication level.
bug #21492: TCP MSS can be larger than MTU
Submitter: | Simon Goldschmidt <goldsimon> | ||
Submitted: | Thu 01 Nov 2007 04:08:28 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: | None |
Wed 21 Nov 2007 06:10:47 PM UTC, comment #4: |
Simon Goldschmidt <goldsimon> |
Wed 14 Nov 2007 07:18:32 AM UTC, comment #3:
|
Simon Goldschmidt <goldsimon> |
Tue 13 Nov 2007 09:16:42 PM UTC, comment #2: I'm probably being dumb, but can you explain the + 20 in this line:
|
Jonathan Larmour <jifl> |
Tue 13 Nov 2007 08:56:50 PM UTC, comment #1: In this patch, I introduced a function tcp_eff_send_mss() that takes a configured mss (the one currently set to pcb->mss in tcp_connect or tcp_listen_input/tcp_process) and calculates the mss_s based on the mtu of the netif used to reach pcb->remote_ip (through ip_route).
|
Simon Goldschmidt <goldsimon> |
Thu 01 Nov 2007 04:08:28 PM UTC, original submission:
This might currently not be a problem for most users, but (according to RFC 1122, chapter 4.2.2.6), the MSS should be the minimum of the received MSS (option) and the 'MTU-minus-headersize'-(calculated)-MSS (effectivly MTU-20 for us).
|
Simon Goldschmidt <goldsimon> |
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 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2007-11-21 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2007-11-13 | goldsimon | Attached File | - | Added effsendmss.patch, #14383 |
Powered by Savane 3.13-f8d8.
Corresponding source code
As noone objects, I'll check this in, together with an option to disable it to reduce code size.