fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-05-08 11:19:16 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

core: fix use-after-free caused by incorrect HAL device resync code

Browse source
This commit is contained in:
Drew Moseley 2009-03-06 17:19:17 -05:00 • committed by Dan Williams
parent 7b45be4c8f
commit 21defe6d73
1 changed files with 8 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/nm-manager.c
View file

@ -1426,12 +1426,11 @@ static void
sync_devices (NMManager *self) sync_devices (NMManager *self)
{ {
NMManagerPrivate *priv = NM_MANAGER_GET_PRIVATE (self); NMManagerPrivate *priv = NM_MANAGER_GET_PRIVATE (self);
GSList *devices; GSList *devices = NULL;
GSList *iter; GSList *iter;
/* Remove devices which are no longer known to HAL */ /* Keep devices still known to HAL; get rid of ones HAL no longer knows about */
devices = g_slist_copy (priv->devices); for (iter = priv->devices; iter; iter = iter->next) {
for (iter = devices; iter; iter = iter->next) {
NMDevice *device = NM_DEVICE (iter->data); NMDevice *device = NM_DEVICE (iter->data);
const char *udi = nm_device_get_udi (device); const char *udi = nm_device_get_udi (device);
@ -1440,15 +1439,14 @@ sync_devices (NMManager *self)
nm_device_set_managed (device, TRUE, NM_DEVICE_STATE_REASON_NOW_MANAGED); nm_device_set_managed (device, TRUE, NM_DEVICE_STATE_REASON_NOW_MANAGED);
else else
nm_device_set_managed (device, FALSE, NM_DEVICE_STATE_REASON_NOW_UNMANAGED); nm_device_set_managed (device, FALSE, NM_DEVICE_STATE_REASON_NOW_UNMANAGED);
} else { devices = g_slist_prepend (devices, device);
priv->devices = g_slist_delete_link (priv->devices, iter); } else
remove_one_device (self, device); remove_one_device (self, device);
}
} }
g_slist_free (priv->devices);
priv->devices = devices;
g_slist_free (devices); /* Ask HAL for new devices */
/* Get any new ones */
nm_hal_manager_query_devices (priv->hal_mgr); nm_hal_manager_query_devices (priv->hal_mgr);
} }