commit ac1328e5143f6ee0054d5cb2f7d17754c16a3814
author Paul Stewart <pstew@chromium.org> Fri Jul 20 11:55:40 2012 -0700
committer Gerrit <chrome-bot@google.com> Mon Jul 30 08:49:00 2012 -0700
tree cd1d1dfa0aa932b2cc1e1d79013a43ecc9c52b4f
parent d001205b0afec0841d72d142f2790de347e52a6c

shill: LinkMonitor: Add ArpClient code

Add ArpClient class and start hooking it up to LinkMonitor.

BUG=chromium-os:32600
TEST=Unit tests.  Some real-world testing using the test harness
to transmit and receive ARP on a real network.

Change-Id: Ic05d8d7eb921878e3776f35b4be285554ef86456
Reviewed-on: https://gerrit.chromium.org/gerrit/28148
Commit-Ready: Paul Stewart <pstew@chromium.org>
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>

arp_packet.cc

diff --git a/arp_packet.cc b/arp_packet.cc
index 5c3998b..6b131ce 100644
--- a/arp_packet.cc
+++ b/arp_packet.cc
@@ -13,6 +13,8 @@
 
 namespace shill {
 
+const size_t ArpPacket::kMinPayloadSize = ETH_ZLEN - ETH_HLEN;
+
 ArpPacket::ArpPacket()
     : local_ip_address_(IPAddress::kFamilyUnknown),
       remote_ip_address_(IPAddress::kFamilyUnknown) {}
@@ -157,6 +159,10 @@
   packet->Append(remote_mac_address_);
   packet->Append(remote_ip_address_.address());
 
+  if (packet->GetLength() < kMinPayloadSize) {
+    packet->Append(ByteString(kMinPayloadSize - packet->GetLength()));
+  }
+
   return true;
 }