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?

Submitted by:  Ron <shifted>
Submitted on:  Tue 10 Jun 2008 03:11:15 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: Faulty BehaviourStatus: Invalid
Privacy: PublicAssigned to: None
Open/Closed: ClosedPlanned Release: None
lwIP version: 1.3.0

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

No worries!

Kieran Mansley <kieranm>
Project Administrator
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>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 10 Jun 2008 07:53:07 AM UTCkieranmOpen/ClosedOpen=>Closed
    Tue 10 Jun 2008 07:53:06 AM UTCkieranmStatusNone=>Invalid

    Back to the top


    Powered by Savane 3.1-cleanup1