lwIP - A Lightweight TCP/IP stack - Bugs: bug #19937, ICMP assumes p_buf has space for...
You are not allowed to post comments on this tracker with your current authentication level.
bug #19937: ICMP assumes p_buf has space for ethernet header
Submitter: | Matthias Weisser <mod42> | ||
Submitted: | Wed 23 May 2007 08:43:30 AM UTC | ||
Category: | IPv4 | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | None |
Jump to the original submission
Fri 15 Jun 2007 06:58:08 AM UTC, comment #9: |
Simon Goldschmidt <goldsimon> |
Thu 14 Jun 2007 05:41:21 PM UTC, comment #8: Checked in the change, works for me now (1000-byte pings, 128-byte pbuf pool size). |
Simon Goldschmidt <goldsimon> |
Thu 14 Jun 2007 06:50:14 AM UTC, comment #7: There seems to be an error here:
|
Simon Goldschmidt <goldsimon> |
Wed 13 Jun 2007 10:42:16 AM UTC, comment #6: OK, closing it as fixed. |
Simon Goldschmidt <goldsimon> |
Tue 12 Jun 2007 06:25:03 AM UTC, comment #5: Today I merged the current CVS version into my port and verified that the change from Simon is working. So from my point of view this bug can be closed.
|
Matthias Weisser <mod42> |
Sat 09 Jun 2007 04:17:35 PM UTC, comment #4:
|
Simon Goldschmidt <goldsimon> |
Sat 09 Jun 2007 12:05:16 PM UTC, comment #3: It's include in my test build since Monday, and I don't have see any problems, but I don't test big pings (and my buffers are large), and it is a "black box" checking...
|
Frédéric Bernon <fbernon> |
Mon 04 Jun 2007 02:31:45 PM UTC, comment #2: Thanks Simon - this is probably best tested by the original reporter of the problem. |
Kieran Mansley <kieranm> |
Sun 03 Jun 2007 11:07:49 AM UTC, comment #1: I've checked in a change to icmp_input: on sending an echo response, p is chacked if it's big enough. If not, a new pbuf (PBUF_RAM) is allocated and used instead. |
Simon Goldschmidt <goldsimon> |
Wed 23 May 2007 08:43:30 AM UTC, original submission:
The ICMP implementation assumes that there is enough space at the front of a pbuf to store an ethernet header. If this is not the case etharp_output fails when trying to add the ethernet header. |
Matthias Weisser <mod42> |
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 |
---|---|---|---|---|---|
2007-06-15 | goldsimon | Status | Ready For Test | Fixed | |
Open/Closed | Open | Closed | |||
2007-06-14 | goldsimon | Status | In Progress | Ready For Test | |
2007-06-14 | goldsimon | Status | Fixed | In Progress | |
Open/Closed | Closed | Open | |||
2007-06-13 | goldsimon | Status | Ready For Test | Fixed | |
Open/Closed | Open | Closed | |||
2007-06-03 | goldsimon | Status | None | Ready For Test | |
Assigned to | None | goldsimon | |||
2007-05-24 | tom_evans | Carbon-Copy | - | Added tom_evans | |
2007-05-23 | mod42 | Carbon-Copy | - | Added -email is unavailable- |
Powered by Savane 3.13-d3ae.
Corresponding source code
Closing this as fixed (again).