RE: [lwip-users] ARP ETHARP_TRY_HARD

[Paul Clarke]

Hi Leon.

The reason why line 494 needs to be for_us = 1;
A few lines later we have

  case ARP_REPLY:
    /* ARP reply. We already updated the ARP cache earlier. */
    LWIP_DEBUGF(ETHARP_DEBUG | DBG_TRACE, ("etharp_arp_input: incoming ARP
reply\n"));
#if (LWIP_DHCP && DHCP_DOES_ARP_CHECK)
    /* DHCP wants to know about ARP replies to our wanna-have-address */
    if (for_us) dhcp_arp_reply(netif, &sipaddr);
#endif
    break;

The other option is to change the code there to

#if (LWIP_DHCP && DHCP_DOES_ARP_CHECK)
    /* DHCP wants to know about ARP replies to our wanna-have-address */
    if (for_us || (netif->ip_addr.addr == 0)) dhcp_arp_reply(netif,
&sipaddr);
#endif

Which is better?

Paul