lwIP - A Lightweight TCP/IP stack - Bugs: bug #28519, lwip_recvfrom bug with len >...
You are not allowed to post comments on this tracker with your current authentication level.
bug #28519: lwip_recvfrom bug with len > 65535
Submitter: | Stephane Lesage <slesage> | ||
Submitted: | Fri 08 Jan 2010 07:01:20 AM UTC | ||
Category: | sockets/netconn | 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
Sat 09 Jan 2010 12:28:35 PM UTC, comment #6: |
Simon Goldschmidt <goldsimon> |
Fri 08 Jan 2010 05:55:23 PM UTC, comment #5: Could size_t be a poor choice for some platforms? Or possibly a type that effects the stack negatively? Your own lwip_size_t would allow it to be different from size_t if that ever were to make sense. |
Bill Auerbach <billauerbach> |
Fri 08 Jan 2010 04:21:28 PM UTC, comment #4: int and size_t could in theory be different. |
Kieran Mansley <kieranm> |
Fri 08 Jan 2010 04:01:53 PM UTC, comment #3:
|
Simon Goldschmidt <goldsimon> |
Fri 08 Jan 2010 02:59:53 PM UTC, comment #2: Should you consider using lwip_size_t which defaults to size_t? Could there be an implementation where size_t isn't optimal? |
Bill Auerbach <billauerbach> |
Fri 08 Jan 2010 02:51:36 PM UTC, comment #1: Fixed using 'int' for 'off' (because return value is 'int', too). Thanks for reporting! |
Simon Goldschmidt <goldsimon> |
Fri 08 Jan 2010 07:01:20 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 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2010-01-08 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2010-01-08 | slesage | Attached File | - | Added lwipdebugout.txt, #19437 |
Powered by Savane 3.13-02a9.
Corresponding source code
But since the return value is int, too, I think it's enough to calculate in int instead of size_t: if the result is too big for int, it will get corrupted when returned, anyway.
It might, but so far, we chose to keep the function definitions according to the standard (as available on www.opengroup.org).