fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2025-12-30 16:40:13 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Revert "dns: don't update routing and DNS if no devices were ever managed"

Browse source 
This reverts commit 0051b44a09.

While fixing up the patch I (dcbw) forgot to include the lp# in
the commit message.
This commit is contained in:
Dan Williams 2011-11-02 15:27:24 -05:00
parent e04cbae154
commit bc8d07a733
2 changed files with 8 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
src/dns-manager/nm-dns-manager.c
View file

@ -1097,20 +1097,15 @@ dispose (GObject *object)
g_slist_free (priv->plugins);
priv->plugins = NULL;
/* If last_iface is NULL, this means we haven't done a DNS update before,
* so no reason to try and take down entries from resolv.conf.
/* If we're quitting leave a valid resolv.conf in place, not one
* pointing to 127.0.0.1 if any plugins were active. Thus update
* DNS after disposing of all plugins.
*/
if (priv->last_iface != NULL) {
/* If we're quitting leave a valid resolv.conf in place, not one
* pointing to 127.0.0.1 if any plugins were active. Thus update
* DNS after disposing of all plugins.
*/
if (!update_dns (self, priv->last_iface, TRUE, &error)) {
nm_log_warn (LOGD_DNS, "could not commit DNS changes on shutdown: (%d) %s",
error ? error->code : -1,
error && error->message ? error->message : "(unknown)");
g_clear_error (&error);
}
if (!update_dns (self, priv->last_iface, TRUE, &error)) {
nm_log_warn (LOGD_DNS, "could not commit DNS changes on shutdown: (%d) %s",
error ? error->code : -1,
error && error->message ? error->message : "(unknown)");
g_clear_error (&error);
}
g_slist_foreach (priv->configs, (GFunc) g_object_unref, NULL);

9
src/nm-policy.c
View file

@ -1042,15 +1042,6 @@ device_state_changed (NMDevice *device,
update_routing_and_dns (policy, FALSE);
break;
case NM_DEVICE_STATE_UNMANAGED:
if ( old_state == NM_DEVICE_STATE_UNAVAILABLE
|| old_state == NM_DEVICE_STATE_DISCONNECTED) {
/* If the device was never activated, there's no point in
* updating routing or DNS. This allows us to keep the previous
* resolv.conf or routes from before NM started if no device was
* ever managed by NM.
*/
break;
}
case NM_DEVICE_STATE_UNAVAILABLE:
update_routing_and_dns (policy, FALSE);
break;