klibc - Mar 2009 - ciaddr in dhcp_proto.c

Hello all:

According to rfc 2131, page 9:
"ciaddr        Client IP address; only filled in if client is in
                  BOUND, RENEW or REBINDING state and can respond
                  to ARP requests.
"

Is it ideal to change "boot.ciaddr = dev-> ip_addr" to
"boot.ciaddr INADDR_ANY" in dhcp_send() function?


I've used a TP-LINK TL-R402M router and it will give a DHCPNAK if
ciaddr is not empty.

Shou-Lien Chen wrote:
> Hello all:
> 
> According to rfc 2131, page 9:
> "ciaddr        Client IP address; only filled in if client is in
>                   BOUND, RENEW or REBINDING state and can respond
>                   to ARP requests.
> "
> 
> Is it ideal to change "boot.ciaddr = dev-> ip_addr" to
"boot.ciaddr > INADDR_ANY" in dhcp_send() function?
> 
> I've used a TP-LINK TL-R402M router and it will give a DHCPNAK if
> ciaddr is not empty.
> 
Makes sense to me, but I'm not a DHCP expert.  Anyone who has more insight?

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.