[lwip-devel] Bug in Checksum implementation and ARP fix

[Paul Riley]

There is a bug in the implementation of lwip_chksum. Basically it assumes that 
you can perform 16 bit access on non 16 bit aligned address 

The offending line is 

    acc += *(u16_t *)dataptr;

This is not valid on all processor architectures.

Before I fix this has anyone else already done this?

In addition I have already fixed a bug in the function update_arp_entry if 
ARPing for an address and hence a packet was queued for later transmission. On 
receipt of the reply the frame is sent, but without a correct source address. 
This is easily fixed with the change shown below

          for (k = 0; k < netif->hwaddr_len; ++k) {
            ethhdr->dest.addr[k] = ethaddr->addr[k];
>>>>>            ethhdr->src.addr[k] = netif->hwaddr[k];
          }

Lastly does anyone know how to get CVS write access. I would like to add in a 
new architecture port?

Thanks,

Paul

P.S. I can send patches in diff -u format if required