X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=src%2Fnetwork%2Fnetworkd-network.c;h=24fc26b92931db81c817776335e1e911eb12679d;hb=b0e39c8284b5195850a58e4efcce2f059bdcc6ad;hp=4e38173140e1d03a2675b0685d8c319c84ab8289;hpb=97578344f3bd03c8ea651d8888622e82c599eadf;p=elogind.git diff --git a/src/network/networkd-network.c b/src/network/networkd-network.c index 4e3817314..24fc26b92 100644 --- a/src/network/networkd-network.c +++ b/src/network/networkd-network.c @@ -23,6 +23,7 @@ #include #include "networkd.h" +#include "networkd-netdev.h" #include "network-internal.h" #include "path-util.h" #include "conf-files.h" @@ -47,8 +48,8 @@ static int network_load_one(Manager *manager, const char *filename) { return -errno; } - if (null_or_empty_path(filename)) { - log_debug("skipping empty file: %s", filename); + if (null_or_empty_fd(fileno(file))) { + log_debug("Skipping empty file: %s", filename); return 0; } @@ -65,10 +66,14 @@ static int network_load_one(Manager *manager, const char *filename) { if (!network->vlans) return log_oom(); - network->macvlans = hashmap_new(uint64_hash_func, uint64_compare_func); + network->macvlans = hashmap_new(string_hash_func, string_compare_func); if (!network->macvlans) return log_oom(); + network->vxlans = hashmap_new(string_hash_func, string_compare_func); + if (!network->vxlans) + return log_oom(); + network->addresses_by_section = hashmap_new(uint64_hash_func, uint64_compare_func); if (!network->addresses_by_section) return log_oom(); @@ -85,13 +90,15 @@ static int network_load_one(Manager *manager, const char *filename) { network->dhcp_dns = true; network->dhcp_hostname = true; network->dhcp_domainname = true; - - r = config_parse(NULL, filename, file, "Match\0Network\0Address\0Route\0DHCPv4\0", config_item_perf_lookup, - (void*) network_network_gperf_lookup, false, false, network); - if (r < 0) { - log_warning("Could not parse config file %s: %s", filename, strerror(-r)); + network->dhcp_routes = true; + network->dhcp_sendhost = true; + + r = config_parse(NULL, filename, file, + "Match\0Network\0Address\0Route\0DHCP\0DHCPv4\0", + config_item_perf_lookup, network_network_gperf_lookup, + false, false, true, network); + if (r < 0) return r; - } LIST_PREPEND(networks, manager->networks, network); @@ -160,21 +167,17 @@ void network_free(Network *network) { free(network->match_name); free(network->description); + free(network->dhcp_vendor_class_identifier); - while ((address = network->ntp)) { - LIST_REMOVE(addresses, network->ntp, address); - address_free(address); - } - - while ((address = network->dns)) { - LIST_REMOVE(addresses, network->dns, address); - address_free(address); - } + strv_free(network->ntp); + strv_free(network->dns); netdev_unref(network->bridge); netdev_unref(network->bond); + netdev_unref(network->tunnel); + HASHMAP_FOREACH(netdev, network->vlans, i) netdev_unref(netdev); hashmap_free(network->vlans); @@ -183,6 +186,10 @@ void network_free(Network *network) { netdev_unref(netdev); hashmap_free(network->macvlans); + HASHMAP_FOREACH(netdev, network->vxlans, i) + netdev_unref(netdev); + hashmap_free(network->vxlans); + while ((route = network->static_routes)) route_free(route); @@ -309,10 +316,11 @@ int config_parse_netdev(const char *unit, break; case NETDEV_KIND_VLAN: - r = hashmap_put(network->vlans, &netdev->vlanid, netdev); + r = hashmap_put(network->vlans, netdev->ifname, netdev); if (r < 0) { log_syntax(unit, LOG_ERR, filename, line, EINVAL, - "Can not add VLAN to network: %s", rvalue); + "Can not add VLAN '%s' to network: %s", + rvalue, strerror(-r)); return 0; } @@ -321,7 +329,18 @@ int config_parse_netdev(const char *unit, r = hashmap_put(network->macvlans, netdev->ifname, netdev); if (r < 0) { log_syntax(unit, LOG_ERR, filename, line, EINVAL, - "Can not add MACVLAN to network: %s", rvalue); + "Can not add MACVLAN '%s' to network: %s", + rvalue, strerror(-r)); + return 0; + } + + break; + case NETDEV_KIND_VXLAN: + r = hashmap_put(network->vxlans, netdev->ifname, netdev); + if (r < 0) { + log_syntax(unit, LOG_ERR, filename, line, EINVAL, + "Can not add VXLAN '%s' to network: %s", + rvalue, strerror(-r)); return 0; }