Re: [lwip-users] ARP Queueing memory leak

[Christiaan Simons]

address@hidden wrote on 30-11-2005
13:07:07:

>
> After getting the latest revision in CVS, the inet.c (line 75) file is
> updated with "dataptr = ((char*)dataptr+2);" is replaced with line "(void
> *)((u16_t *)dataptr + 1);"
>
> This seems to be a bug or?
>

We've noticed this may be bug.

I was just fiddling to get it in
a workable state for the 16 bit c16x mcu.

The void pointer increment was clearly wrong,
the u16_t increment is only correct for aligned
input data.

The "two char" increment looks more correct,
but the routine will still fail for the c16x.

I'm a bit lost on how to get the checksumming
right for all architectures, both little-
an big-endian, 16-bit aligned and unaligned access.

The best thing you can do is stick with the lwip
release 1.1.0 for now.


Christiaan Simons

Hardware / Software Engineer
Axon Digital Design

+31 (0)13 511 66 66
+31 (0)13 511 41 51

http://www.axon.tv



This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error please notify the system manager.
This message contains confidential information and is intended only for the
individual named.  If you are not the named addressee you should not
disseminate, distribute or copy this e-mail.