struct interface *iface;
struct device *dev;
struct device_addr *addr;
+ bool keep = false;
ip = container_of(tree, struct interface_ip_settings, addr);
iface = ip->iface;
dev = iface->l3_dev->dev;
+ if (node_old && node_new)
+ keep = true;
+
if (node_old) {
addr = container_of(node_old, struct device_addr, node);
- if (!(addr->flags & DEVADDR_EXTERNAL) && addr->enabled)
+ if (!(addr->flags & DEVADDR_EXTERNAL) && addr->enabled && !keep)
system_del_address(dev, addr);
free(addr);
}
if (node_new) {
addr = container_of(node_new, struct device_addr, node);
- if (!(addr->flags & DEVADDR_EXTERNAL))
+ if (!(addr->flags & DEVADDR_EXTERNAL) && !keep)
system_add_address(dev, addr);
addr->enabled = true;
}
struct interface_ip_settings *ip;
struct interface *iface;
struct device *dev;
- struct device_route *route;
+ struct device_route *route_old, *route_new;
+ bool keep = false;
ip = container_of(tree, struct interface_ip_settings, route);
iface = ip->iface;
dev = iface->l3_dev->dev;
+ route_old = container_of(node_old, struct device_route, node);
+ route_new = container_of(node_new, struct device_route, node);
+
+ if (node_old && node_new)
+ keep = !memcmp(&route_old->nexthop, &route_new->nexthop, sizeof(route_old->nexthop));
+
if (node_old) {
- route = container_of(node_old, struct device_route, node);
- if (!(route->flags & DEVADDR_EXTERNAL) && route->enabled)
- system_del_route(dev, route);
- free(route);
+ if (!(route_old->flags & DEVADDR_EXTERNAL) && route_old->enabled && !keep)
+ system_del_route(dev, route_old);
+ free(route_old);
}
if (node_new) {
- route = container_of(node_new, struct device_route, node);
- if (!(route->flags & DEVADDR_EXTERNAL))
- system_add_route(dev, route);
- route->enabled = true;
+ if (!(route_new->flags & DEVADDR_EXTERNAL) && !keep)
+ system_add_route(dev, route_new);
+ route_new->enabled = true;
}
}
add:
D(INTERFACE, "Add IPv%c DNS server: %s\n",
s->af == AF_INET6 ? '6' : '4', str);
- list_add_tail(&s->list, &ip->dns_servers);
+ vlist_simple_add(&ip->dns_servers, &s->node);
}
void
D(INTERFACE, "Add DNS search domain: %s\n", str);
memcpy(s->name, str, len);
- list_add_tail(&s->list, &ip->dns_search);
+ vlist_simple_add(&ip->dns_search, &s->node);
}
void
}
static void
-interface_clear_dns_servers(struct interface_ip_settings *ip)
+write_resolv_conf_entries(FILE *f, struct interface_ip_settings *ip)
{
- struct dns_server *s, *tmp;
+ struct dns_server *s;
+ struct dns_search_domain *d;
+ const char *str;
+ char buf[32];
- list_for_each_entry_safe(s, tmp, &ip->dns_servers, list) {
- list_del(&s->list);
- free(s);
- }
-}
+ vlist_simple_for_each_element(&ip->dns_servers, s, node) {
+ str = inet_ntop(s->af, &s->addr, buf, sizeof(buf));
+ if (!str)
+ continue;
-static void
-interface_clear_dns_search(struct interface_ip_settings *ip)
-{
- struct dns_search_domain *s, *tmp;
+ fprintf(f, "nameserver %s\n", str);
+ }
- list_for_each_entry_safe(s, tmp, &ip->dns_search, list) {
- list_del(&s->list);
- free(s);
+ vlist_simple_for_each_element(&ip->dns_search, d, node) {
+ fprintf(f, "search %s\n", d->name);
}
}
interface_write_resolv_conf(void)
{
struct interface *iface;
- struct dns_server *s;
- struct dns_search_domain *d;
char *path = alloca(strlen(resolv_conf) + 5);
- const char *str;
- char buf[32];
FILE *f;
sprintf(path, "%s.tmp", resolv_conf);
if (iface->state != IFS_UP)
continue;
- if (list_empty(&iface->proto_ip.dns_search) &&
- list_empty(&iface->proto_ip.dns_servers))
+ if (vlist_simple_empty(&iface->proto_ip.dns_search) &&
+ vlist_simple_empty(&iface->proto_ip.dns_servers) &&
+ vlist_simple_empty(&iface->config_ip.dns_search) &&
+ vlist_simple_empty(&iface->config_ip.dns_servers))
continue;
fprintf(f, "# Interface %s\n", iface->name);
- list_for_each_entry(s, &iface->proto_ip.dns_servers, list) {
- str = inet_ntop(s->af, &s->addr, buf, sizeof(buf));
- if (!str)
- continue;
-
- fprintf(f, "nameserver %s\n", str);
- }
-
- list_for_each_entry(d, &iface->proto_ip.dns_search, list) {
- fprintf(f, "search %s\n", d->name);
- }
+ write_resolv_conf_entries(f, &iface->config_ip);
+ write_resolv_conf_entries(f, &iface->proto_ip);
}
fclose(f);
if (rename(path, resolv_conf) < 0) {
void
interface_ip_update_start(struct interface_ip_settings *ip)
{
- interface_clear_dns_servers(ip);
- interface_clear_dns_search(ip);
+ vlist_simple_update(&ip->dns_servers);
+ vlist_simple_update(&ip->dns_search);
vlist_update(&ip->route);
vlist_update(&ip->addr);
}
void
interface_ip_update_complete(struct interface_ip_settings *ip)
{
+ vlist_simple_flush(&ip->dns_servers);
+ vlist_simple_flush(&ip->dns_search);
vlist_flush(&ip->route);
vlist_flush(&ip->addr);
}
void
interface_ip_flush(struct interface_ip_settings *ip)
{
- interface_clear_dns_servers(ip);
- interface_clear_dns_search(ip);
+ vlist_simple_flush_all(&ip->dns_servers);
+ vlist_simple_flush_all(&ip->dns_search);
vlist_flush_all(&ip->route);
vlist_flush_all(&ip->addr);
}
{
ip->iface = iface;
ip->enabled = true;
- INIT_LIST_HEAD(&ip->dns_search);
- INIT_LIST_HEAD(&ip->dns_servers);
+ vlist_simple_init(&ip->dns_search, struct dns_search_domain, node);
+ vlist_simple_init(&ip->dns_servers, struct dns_server, node);
vlist_init(&ip->route, route_cmp, interface_update_proto_route,
struct device_route, node, mask);
vlist_init(&ip->addr, addr_cmp, interface_update_proto_addr,