lwIP - A Lightweight TCP/IP stack - Bugs: bug #29769, sys_check_timeouts: sys_now() may...
You are not allowed to post comments on this tracker with your current authentication level.
bug #29769: sys_check_timeouts: sys_now() may overflow
Submitter: | Simon Goldschmidt <goldsimon> | ||
Submitted: | Tue 04 May 2010 10:26:23 AM UTC | ||
Category: | None | Severity: | 3 - Normal |
Item Group: | None | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | CVS Head |
Tue 04 May 2010 07:29:01 PM UTC, comment #2: |
Simon Goldschmidt <goldsimon> |
Tue 04 May 2010 01:39:19 PM UTC, comment #1: I use the following macro:
|
Iordan Neshev <iordan_neshev> |
Tue 04 May 2010 10:26:23 AM UTC, original submission:
For NO_SYS==1, sys_check_timeouts uses sys_now() time-difference to check for timeouts. Now when the time wraps around (after 49-something days), no more timers will get called. Possible solution:
|
Simon Goldschmidt <goldsimon> |
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 3 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2010-05-04 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed |
Powered by Savane 3.13-d3ae.
Corresponding source code
That should work on any architecture since sys_now() and 'last_time' both give u32_t values.
Fixed by adding this macro as 'LWIP_U32_DIFF' to def.h.
Thanks for the code ;-)