commit 05368f128351c8aef191f3c30af8942eb7041b67
author William A. Kennington III <wak@google.com> Thu May 13 18:40:47 2021 -0700
committer William A. Kennington III <wak@google.com> Thu May 13 18:41:52 2021 -0700
tree fa6851fce9940c6d203dbd3af762725e68b185d2
parent 6f39c5e25ea112de313badbf3a217f609ec6a2d0

sources: Fully initialize structs

Some of our declarations only initialize part of a struct or value, use
an empty initializer statement to fully initialize them. For some with
non-default values, make sure the other values in the struct are
populated.

Change-Id: Ieee9126267ad318fde071a231aa33bb52501b15f
Signed-off-by: William A. Kennington III <wak@google.com>

ethernet_interface.cpp

diff --git a/ethernet_interface.cpp b/ethernet_interface.cpp
index 2de60e4..ce424ff 100644
--- a/ethernet_interface.cpp
+++ b/ethernet_interface.cpp
@@ -357,13 +357,13 @@
 */
 InterfaceInfo EthernetInterface::getInterfaceInfo() const
 {
-    ifreq ifr{0};
-    ethtool_cmd edata{0};
-    LinkSpeed speed{0};
-    Autoneg autoneg{0};
-    DuplexMode duplex{0};
-    LinkUp linkState{false};
-    NICEnabled enabled{false};
+    ifreq ifr = {};
+    ethtool_cmd edata = {};
+    LinkSpeed speed = {};
+    Autoneg autoneg = {};
+    DuplexMode duplex = {};
+    LinkUp linkState = {};
+    NICEnabled enabled = {};
     EthernetIntfSocket eifSocket(PF_INET, SOCK_DGRAM, IPPROTO_IP);
 
     if (eifSocket.sock < 0)
@@ -404,7 +404,7 @@
         return activeMACAddr;
     }
 
-    ifreq ifr{0};
+    ifreq ifr = {};
     std::strncpy(ifr.ifr_name, interfaceName.c_str(), IFNAMSIZ - 1);
     if (ioctl(eifSocket.sock, SIOCGIFHWADDR, &ifr) != 0)
     {
@@ -579,7 +579,7 @@
         return value;
     }
 
-    ifreq ifr{0};
+    ifreq ifr = {};
     std::strncpy(ifr.ifr_name, interfaceName().c_str(), IF_NAMESIZE - 1);
     if (ioctl(eifSocket.sock, SIOCGIFFLAGS, &ifr) == 0)
     {
@@ -603,7 +603,7 @@
         return value;
     }
 
-    ifreq ifr{0};
+    ifreq ifr = {};
     std::strncpy(ifr.ifr_name, interfaceName().c_str(), IF_NAMESIZE - 1);
     if (ioctl(eifSocket.sock, SIOCGIFFLAGS, &ifr) == 0)
     {
@@ -630,7 +630,7 @@
         return EthernetInterfaceIntf::nicEnabled();
     }
 
-    ifreq ifr{0};
+    ifreq ifr = {};
     std::strncpy(ifr.ifr_name, interfaceName().c_str(), IF_NAMESIZE - 1);
     if (ioctl(eifSocket.sock, SIOCGIFFLAGS, &ifr) != 0)
     {