Commit 3498d922 authored by Jon Siwek's avatar Jon Siwek

Fix a file descriptor leak in get_mac_addresses()

parent 1d632b76
......@@ -110,6 +110,7 @@ std::vector<iface_info> get_mac_addresses() {
ifc.ifc_buf = buf;
if (ioctl(sck, SIOCGIFCONF, &ifc) < 0) {
perror("ioctl(SIOCGIFCONF)");
close(sck);
return {};
}
std::vector<iface_info> result;
......@@ -124,6 +125,7 @@ std::vector<iface_info> get_mac_addresses() {
// get mac address
if (ioctl(sck, SIOCGIFHWADDR, item) < 0) {
perror("ioctl(SIOCGIFHWADDR)");
close(sck);
return {};
}
std::ostringstream oss;
......@@ -140,6 +142,7 @@ std::vector<iface_info> get_mac_addresses() {
result.push_back({item->ifr_name, std::move(addr)});
}
}
close(sck);
return result;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment