From: Shawn Landden <shawn@churchofgit.com>
Date: Wed, 11 Mar 2015 03:45:15 +0000 (-0700)
Subject: sd-dhcp-client: fix strict aliasing issue
X-Git-Tag: v219.0~311
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=6ec8e7c763b7dfa82e25e31f6938122748d1608f;ds=sidebyside

sd-dhcp-client: fix strict aliasing issue
---

diff --git a/src/libsystemd-network/sd-dhcp-client.c b/src/libsystemd-network/sd-dhcp-client.c
index 4224e0197..a477cccec 100644
--- a/src/libsystemd-network/sd-dhcp-client.c
+++ b/src/libsystemd-network/sd-dhcp-client.c
@@ -1469,7 +1469,7 @@ static int client_receive_message_udp(sd_event_source *s, int fd,
         _cleanup_free_ DHCPMessage *message = NULL;
         int buflen = 0, len, r;
         const struct ether_addr zero_mac = { { 0, 0, 0, 0, 0, 0 } };
-        const struct ether_addr *expected_chaddr = NULL;
+        bool expect_chaddr;
         uint8_t expected_hlen = 0;
 
         assert(s);
@@ -1514,11 +1514,11 @@ static int client_receive_message_udp(sd_event_source *s, int fd,
 
         if (client->arp_type == ARPHRD_ETHER) {
                 expected_hlen = ETH_ALEN;
-                expected_chaddr = (const struct ether_addr *) &client->mac_addr;
+                expect_chaddr = true;
         } else {
                /* Non-ethernet links expect zero chaddr */
                expected_hlen = 0;
-               expected_chaddr = &zero_mac;
+               expect_chaddr = false;
         }
 
         if (message->hlen != expected_hlen) {
@@ -1526,7 +1526,10 @@ static int client_receive_message_udp(sd_event_source *s, int fd,
                 return 0;
         }
 
-        if (memcmp(&message->chaddr[0], expected_chaddr, ETH_ALEN)) {
+        if (memcmp(&message->chaddr[0], expect_chaddr ?
+                                          (void *)&client->mac_addr :
+                                          (void *)&zero_mac,
+                                        ETH_ALEN)) {
                 log_dhcp_client(client, "received chaddr does not match "
                                 "expected: ignoring");
                 return 0;