buglwIP - A Lightweight TCP/IP stack - Bugs: bug #23532, Bug in MEM.h for memory size?

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #23532: Bug in MEM.h for memory size?

Submitter:  Ron <shifted>
Submitted:  Tue 10 Jun 2008 03:11:15 AM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Invalid
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Planned Release:  None
lwIP version:  1.3.0

Tue 10 Jun 2008 07:53:07 AM UTC, comment #2: 

No worries!

Kieran Mansley <kieranm>
Group Member
Tue 10 Jun 2008 03:12:42 AM UTC, comment #1: 

Nevermind, my font is set so that the lower case L and 1 look exactly the same, my bad, apologies.


Ron <shifted>
Tue 10 Jun 2008 03:11:15 AM UTC, original submission:  

I think the following lines in the includes/lwip/mem.h file that are currently:

/* MEM_SIZE would have to be aligned, but using 64000 here instead of

  • 65535 leaves some room for alignment...

 */
#if MEM_SIZE > 64000l
typedef u32_t mem_size_t;
#else
typedef u16_t mem_size_t;
#endif /* MEM_SIZE > 64000 */

Should read to be #if MEM_SIZE > 64001, not 640001. 640000 won't fit into an unsigned short.

This bug is on line 66 of mem.h.


Ron <shifted>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by kieranm (Posted a comment)
  • -email is unavailable- added by shifted (Submitted the item)
  •  

    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 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2008-06-10 kieranm Open/ClosedOpen Closed
    2008-06-10 kieranm StatusNone Invalid

    Back to the top

    Powered by Savane 3.13-aa77.
    Corresponding source code