fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-01-07 03:10:16 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

wifi/supplicant: take object reference during availability_changed()

Browse source 
If the list is expected to be modified, it also means that possibly
instances will be unrefed. Probably, not yet visited instances will
not be unrefed so there is no real problem.

Just be extra cautious and take a reference to all instances first.

(cherry picked from commit 004edecc81)
This commit is contained in:
Thomas Haller 2016-01-19 14:30:12 +01:00
parent 91bdabc808
commit 213c20142c
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/supplicant-manager/nm-supplicant-manager.c
View file

@ -200,9 +200,11 @@ availability_changed (NMSupplicantManager *self, gboolean available)
/* priv->ifaces may be modified if availability changes; can't use GHashTableIter */
ifaces = g_hash_table_get_values (priv->ifaces);
for (iter = ifaces; iter; iter = iter->next)
g_object_ref (iter->data);
for (iter = ifaces; iter; iter = iter->next)
nm_supplicant_interface_set_supplicant_available (NM_SUPPLICANT_INTERFACE (iter->data), available);
g_list_free (ifaces);
g_list_free_full (ifaces, g_object_unref);
}
static gboolean