lwIP - A Lightweight TCP/IP stack - Bugs: bug #39683, Assertion "seg->tcphdr not...
You are not allowed to post comments on this tracker with your current authentication level.
bug #39683: Assertion "seg->tcphdr not aligned" failed with MEM_ALIGNMENT = 8
Submitter: | Christian Prochaska <cproc> | ||
Submitted: | Sat 03 Aug 2013 08:17:30 PM UTC | ||
Category: | TCP | Severity: | 3 - Normal |
Item Group: | None | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | git head |
Jump to the original submission
Mon 11 May 2015 09:15:30 AM UTC, comment #7: |
Sylvain Rochet <gradator> |
Sun 10 May 2015 10:38:08 PM UTC, comment #6: I've been using
|
Valery Ushakov <uwe> |
Fri 08 May 2015 05:33:19 PM UTC, comment #5: It also breaks things for MEM_ALIGNMENT==1, MEM_ALIGNMENT is historically set to 1 in unix port minimal example by default.
|
Sylvain Rochet <gradator> |
Wed 01 Apr 2015 11:35:39 AM UTC, comment #4: The proposed/applied fix breaks things for MEM_ALIGNMENT==2. |
Simon Goldschmidt <goldsimon> |
Thu 20 Feb 2014 09:35:56 PM UTC, comment #3: Fixed, thanks. |
Simon Goldschmidt <goldsimon> |
Tue 17 Sep 2013 07:09:31 PM UTC, comment #2:
|
Christian Prochaska <cproc> |
Sun 08 Sep 2013 09:23:35 PM UTC, comment #1: We have a similar problem here, and I just wanted to file a similar bug.
|
Thomas Kindler <thomask77> |
Sat 03 Aug 2013 08:17:30 PM UTC, original submission:
'seg->tcphdr' often gets calculated as 'p->payload - TCP_HLEN', where 'p->payload' is aligned to MEM_ALIGNMENT and TCP_HLEN is a constant 20. So, with MEM_ALIGNMENT = 8 (for x86_64), the result cannot be aligned to MEM_ALIGNMENT.
|
Christian Prochaska <cproc> |
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 8 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2015-05-11 | gradator | Status | In Progress | Fixed | |
Open/Closed | Open | Closed | |||
2015-04-01 | goldsimon | Status | Fixed | In Progress | |
Open/Closed | Closed | Open | |||
2014-02-20 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2013-09-08 | thomask77 | Carbon-Copy | - | Added -email is unavailable- |
Powered by Savane 3.13-f8d8.
Corresponding source code
I guess you are right, LWIP_MIN(MEM_ALIGNMENT, 4) should do the trick.