pgrondek/polybar-dwm
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix(network): Check for nullptr values

Browse source 
Ref #159
This commit is contained in:
Michael Carlberg 2016-11-14 14:25:18 +01:00
parent 983963d4e4
commit 7b293a42cd
1 changed files with 8 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/adapters/net.cpp
View file

@ -62,7 +62,7 @@ namespace net {
bool network::query(bool accumulate) {
struct ifaddrs* ifaddr;
if (getifaddrs(&ifaddr) == -1)
if (getifaddrs(&ifaddr) == -1 || ifaddr == nullptr)
return false;
m_status.previous = m_status.current;
@ -71,11 +71,12 @@ namespace net {
m_status.current.time = chrono::system_clock::now();
for (auto ifa = ifaddr; ifa != nullptr; ifa = ifa->ifa_next) {
if (m_interface.compare(0, m_interface.length(), ifa->ifa_name) != 0) {
if (!accumulate || ifa->ifa_addr->sa_family != AF_PACKET) {
if (ifa->ifa_data == nullptr || ifa->ifa_addr == nullptr)
continue;
if (m_interface.compare(0, m_interface.length(), ifa->ifa_name) != 0)
if (!accumulate || ifa->ifa_addr->sa_family != AF_PACKET)
continue;
}
}
switch (ifa->ifa_addr->sa_family) {
case AF_INET:
@ -85,9 +86,9 @@ namespace net {
break;
case AF_PACKET:
if (ifa->ifa_data == nullptr)
continue;
struct rtnl_link_stats* link_state = reinterpret_cast<decltype(link_state)>(ifa->ifa_data);
if (link_state == nullptr)
continue;
m_status.current.transmitted += link_state->tx_bytes;
m_status.current.received += link_state->rx_bytes;
break;