tree e0d0af861c9465e8e66b437355a0dad08ebc129b
parent 3a40b0deabe95b74d7eb3d4afbef09c34012a660
author Lorenzo Colitti <lorenzo@google.com> 1433234784 +0900
committer Lorenzo Colitti <lorenzo@google.com> 1433397431 +0900

Actually fall back from yiaddr to ciaddr.

The initial implementation of toDhcpResults attempted to get the
leased IP address from ciaddr if yiaddr was 0.0.0.0, but it never
actually did so because a) it used == instead of equals(), and b)
the parsing code never populated mClientIp for a DhcpOfferPacket
or DhcpAckPacket.

Fix this and add a test for it.. Technically DHCP does not use
ciaddr (only bootp uses it), but in 5.0 we would use ciaddr if
yiaddr was 0.0.0.0 and a bit more compatibility shouldn't hurt.

Bug: 19704592
Change-Id: I1f58555f0c10b9c576995a6edb759a83d8938ea0