HM2 wrote:
This is probably nit picky, but I think the Target Hardware Address should
be set to the FF... broadcast value, rather than Zero.  This is in Etharp.c
in the function etharp_request().  &ethzero is used where I think it should
be &ethbroadcast.  In the other examples and doc I always see  FF... used
for this value.
In chapter 4.6, RFC 3220 says

        When using an ARP Reply packet,
        the Target Hardware Address is also set to the link-layer
        address to which this cache entry should be updated (this field
        is not used in an ARP Request packet).

Since we use a request packet, it is OK to use ethzero (in fact, it doesn't 
matter what we use).

Simon



_______________________________________________
lwip-users mailing list
[email protected]
http://lists.nongnu.org/mailman/listinfo/lwip-users

Reply via email to