lwIP - A Lightweight TCP/IP stack - Bugs: bug #28798, IGMP "Max Response Time"...
You are not allowed to post comments on this tracker with your current authentication level.
bug #28798: IGMP "Max Response Time" processing bug
Submitter: | Stephane Lesage <slesage> | ||
Submitted: | Wed 03 Feb 2010 10:32:08 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: | CVS Head |
Jump to the original submission
Mon 08 Feb 2010 07:52:06 PM UTC, comment #9: |
Simon Goldschmidt <goldsimon> |
Mon 08 Feb 2010 07:19:23 PM UTC, comment #8:
|
Stephane Lesage <slesage> |
Mon 08 Feb 2010 06:47:09 PM UTC, comment #7:
|
Simon Goldschmidt <goldsimon> |
Mon 08 Feb 2010 06:31:38 PM UTC, comment #6:
|
Stephane Lesage <slesage> |
Mon 08 Feb 2010 06:14:38 PM UTC, comment #5: I've fixed the actual bug but didn't yet check in the stats changes.
|
Simon Goldschmidt <goldsimon> |
Mon 08 Feb 2010 05:46:20 PM UTC, comment #4: Could you please separate coding style functions from bugfixes from other improvements next time? It's more time consuming to analyse what you changed and why if you send one big patch with all your changes! |
Simon Goldschmidt <goldsimon> |
Thu 04 Feb 2010 06:24:57 PM UTC, comment #3: Here are patches against the current CVS head for:
|
Stephane Lesage <slesage> |
Thu 04 Feb 2010 10:57:45 AM UTC, comment #2: Thanks for reporting and suggesting a fix.
|
Kieran Mansley <kieranm> |
Wed 03 Feb 2010 04:46:08 PM UTC, comment #1: Actually the comparison alone is wrong, because the timer can be inactive at 0, so we must also compare to 0.
|
Stephane Lesage <slesage> |
Wed 03 Feb 2010 10:32:08 AM UTC, original submission:
Hi,
|
Stephane Lesage <slesage> |
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 9 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2010-02-08 | goldsimon | Status | In Progress | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2010-02-08 | goldsimon | Status | None | In Progress | |
2010-02-04 | slesage | Attached File | - | Added igmp.h.diff, #19620 | |
Attached File | - | Added igmp.c.diff, #19621 | |||
Attached File | - | Added stats.h.diff, #19622 | |||
Attached File | - | Added stats.c.diff, #19623 | |||
2010-02-03 | slesage | Attached File | - | Added igmp.c, #19609 |
Powered by Savane 3.13-02a9.
Corresponding source code
Fixed. Thanks for the input. I've internalized many functions and defines, too, in order to cleanly separate the usage. I'd be happy if you could have a look at the result, only to verify I didn't internalize too much...